Skip to content

Create All the most important terms from A-Z - #26

Open
Nefn-Lakhouj wants to merge 5 commits into
mainfrom
Nefn-Lakhouj
Open

Create All the most important terms from A-Z#26
Nefn-Lakhouj wants to merge 5 commits into
mainfrom
Nefn-Lakhouj

Conversation

@Nefn-Lakhouj

Copy link
Copy Markdown

No description provided.

@gelilaa
gelilaa self-requested a review October 9, 2021 14:51

@gelilaa gelilaa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Nefn-Lakhouj can you please fix the conflict?

@Nefn-Lakhouj

Copy link
Copy Markdown
Author

@gelilaa i fix the cconfict
is it ok now?

@arnochauveau arnochauveau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments

Comment thread admin/seeds/README.md
@@ -1,107 +0,0 @@
# About Repository

@arnochauveau arnochauveau Dec 6, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't delete files please :-)
You can re-add this file by executing git checkout origin main -- admin/seeds/README.md

@@ -0,0 +1,568 @@
# All the most important terms from A-Z

@arnochauveau arnochauveau Dec 6, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file has no extension (should be a MD file) it's also bad practice to have spaces in filenames.
Also: all the newlines make it really hard to read.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants