Skip to content

docs: enhance CONTRIBUTING.md#1262

Merged
ematipico merged 1 commit intowebpack:nextfrom
snitin315:chore/improve-contributing.md
Feb 24, 2020
Merged

docs: enhance CONTRIBUTING.md#1262
ematipico merged 1 commit intowebpack:nextfrom
snitin315:chore/improve-contributing.md

Conversation

@snitin315
Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

Made some enhancements in CONTRIBUTING.md .

Did you add tests for your changes?
NA
If relevant, did you update the documentation?
Yes
Summary
Screenshot from 2020-02-24 20-29-33

Screenshot from 2020-02-24 20-29-19

Does this PR introduce a breaking change?

No

@webpack-bot
Copy link
Copy Markdown

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

@snitin315 snitin315 force-pushed the chore/improve-contributing.md branch from b07b9f1 to ea32ae3 Compare February 24, 2020 15:22
@snitin315
Copy link
Copy Markdown
Member Author

@ematipico Updated 👍

Copy link
Copy Markdown
Contributor

@ematipico ematipico left a comment

Choose a reason for hiding this comment

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

Thank you very much for this contribution! I would like to add a new section called "Testing the CLI locally". I can create an issue there and explain what's on my mind

@webpack-bot
Copy link
Copy Markdown

@snitin315 Thanks for your update.

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

@ematipico Please review the new changes.

Copy link
Copy Markdown
Member

@anshumanv anshumanv left a comment

Choose a reason for hiding this comment

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

lgtm nice

Copy link
Copy Markdown
Contributor

@wizardofhogwarts wizardofhogwarts left a comment

Choose a reason for hiding this comment

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

Lgtm

@ematipico ematipico merged commit f1b7269 into webpack:next Feb 24, 2020
anshumanv pushed a commit to anshumanv/webpack-cli that referenced this pull request Feb 27, 2020
@snitin315 snitin315 deleted the chore/improve-contributing.md branch March 21, 2020 05:34
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.

6 participants