Skip to content

docs(contributing): adds prebuild instructions#757

Merged
evenstensberg merged 4 commits into
webpack:masterfrom
misterdev:patch-4
Mar 8, 2019
Merged

docs(contributing): adds prebuild instructions#757
evenstensberg merged 4 commits into
webpack:masterfrom
misterdev:patch-4

Conversation

@misterdev

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

Happy to help!

@dhruvdutt

dhruvdutt commented Feb 27, 2019

Copy link
Copy Markdown
Contributor

@misterdev Can you please sign the CLA? 😅

@mikesherov

Copy link
Copy Markdown

You mean @misterdev

@misterdev

misterdev commented Feb 27, 2019

Copy link
Copy Markdown
Contributor Author

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

Anyway I've renewed my signature

@webpack-bot

Copy link
Copy Markdown

@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
Copy Markdown
Contributor Author

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

@evenstensberg evenstensberg left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
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