Skip to content

Conversation

@misterdev
Copy link
Contributor

Adds few lines to CONTRIBUTING.md about the undocumented boostrap npm script that should be run to set up the project

@TheLarkInn
Copy link
Member

Wow thanks a bunch for this! I was looking for this myself earlier and didn't even realize it wasn't in CONTRIBUTING.md

@misterdev
Copy link
Contributor Author

Happy to help!

@dhruvdutt
Copy link
Contributor

dhruvdutt commented Feb 27, 2019

@misterdev Can you please sign the CLA? 😅

@mikesherov
Copy link

You mean @misterdev

@misterdev
Copy link
Contributor Author

misterdev commented Feb 27, 2019

I see it is already signed :)
screenshot 2019-02-27 at 17 01 56

Anyway I've renewed my signature

@webpack-bot
Copy link

@misterdev Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@dhruvdutt Please review the new changes.

@misterdev
Copy link
Contributor Author

I've indented those instructions to make them even with #768

Copy link
Member

@evenstensberg evenstensberg left a comment

Choose a reason for hiding this comment

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

lgtm, thanks for the fix

@evenstensberg
Copy link
Member

We're currently freezing PR's until our tests are fixed, sorry for the delay

@evenstensberg evenstensberg merged commit 637c47c into webpack:master Mar 8, 2019
@misterdev misterdev deleted the patch-4 branch March 22, 2019 16:42
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.

7 participants