-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
adding my art to the project #2584
Conversation
🙏 Thanks for your pull request @codercreative, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated! Some of the most popular are PR Statistics
|
🙏 Thanks for your pull request @codercreative, The team will now review and merge this request. In the mean time why not check out some of the other opensource projects available, contributions are greatly appreciated! Some of the most popular are PR Statistics
|
Removed useless gitignore from within
|
1 similar comment
|
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.
Update your repository ( modifications has been done ) , solve the conflicts --> then address the following feedbacks :)
Please change the folder name:
- folder name: it should be named after your github user name,
- include.js: once done: adjust the include.js so the path are adjusted accordiningly
|
|
Feedback - Please avoid pushing multiple time a MR - the previous merge request is / will be closed - your change should be on this branch. Thanks |
Please add me to this list.