-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Frist contribution #3598
Frist contribution #3598
Conversation
This repository is for practice only and therefore it will not count towards hacktoberfest. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your name is not in the correct alphabetical order.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
your name should in fact be between those two:
- [Godfred Doe](https://github.com/FredDoe)
- [Gokil P](https://github.com/Gokilp)
- [Gokulraj Puviyarasu](https://github.com/gokulrajpuviyarasu)
please merge my PR |
@@ -1054,7 +1054,6 @@ A GitHub conflict is when people make changes to the same area or line in a file | |||
| [`Back To Top`](#contents) | | |||
|
|||
- ### **G** | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't remove this empty line as it breaks the consistency for other alphabets
Update README.md
Related issue
What changes does this PR do?
Screenshots (If any)
Questions