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
Add webpack-cli and jest to project list #33
Conversation
@bnb Please review this. |
lib/projects.js
Outdated
@@ -41,6 +41,10 @@ var projects = { | |||
"webpack-cli": { | |||
name: 'webpack-cli', |
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.
Since the module on npm uses "webpack CLI" let's use that 😄
name: 'webpack-cli', | |
name: 'webpack CLI', |
lib/projects.js
Outdated
@@ -41,6 +41,10 @@ var projects = { | |||
"webpack-cli": { | |||
name: 'webpack-cli', | |||
q: 'repo:webpack/webpack-cli is:open is:issue label:"Good First Contribution"' | |||
}, | |||
jest: { | |||
name: 'jest', |
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.
Same goes for the Jest website, using "Jest" rather than the lower-case "jest".
name: 'jest', | |
name: 'Jest', |
Otherwise LGTM and this can be merged once those changes are made – thanks for the PR! 👍 |
@hemal7735 please rebase and resolve the conflicts |
LGTM! Thank you for the PR @hemal7735 💥 |
No description provided.