-
Notifications
You must be signed in to change notification settings - Fork 156
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
Submit to the curated list of awesome lists? #48
Comments
I tried adding it to their list, which would have made it more discoverable, but their requirements are quite strict and this list is failing their lint miserably. While I can fix a few things, only maintainers (@ruebot & @anjackson ) of the repo can do the rest.
|
@ibnesayeed feel free to take it on, and let me or @anjackson know what needs to be changed admin-wise. |
I will sync my local repo after #74 is merged then will try to fix some of these lint issues. |
|
I found out that the license is already CC0, but it was inline, not as a separate file, so I am going to add it. |
Done. |
Great to see progress on this! Thanks to everybody who contributed time :) |
Submitted to the awesome list for inclusion. sindresorhus/awesome#1704 |
Please open this ticket as the PR is not merged in the meta list. |
@ibnesayeed you had an actionable keyword in your PR. It closed because of you. If you did not intend that, please don't use them.
|
Yes, it was an oversight on my end, I should have phrased the commit message better. |
sindresorhus/awesome#1704 is merged and tweeted so this issue can be closed now. |
I suggest this list should be submitted to https://github.com/sindresorhus/awesome
Here are the instructions: https://github.com/sindresorhus/awesome/blob/master/pull_request_template.md
The text was updated successfully, but these errors were encountered: