Skip to content
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

Updated working link for webpack addon. #543

Merged
merged 1 commit into from
Jul 13, 2018

Conversation

NaviMarella
Copy link
Contributor

@NaviMarella NaviMarella commented Jul 11, 2018

What kind of change does this PR introduce?
The link provided in webpack scaffold your package section is missing. I have updated the link, Please verify and accept this PR.

Did you add tests for your changes?
As this is a documentation change, I assume no tests are required.

If relevant, did you update the documentation?

Summary
Trying to help with documentation.

Does this PR introduce a breaking change?
No, it doesn't introduce a breaking change.

Other information
This is my first PR to open-source and i am really excited to contribute to webpack.

The link provided in [ webpack-scaffold-yourpackage ] section is missing. I have updated the link, Please verify and accept this PR.
@jsf-clabot
Copy link

jsf-clabot commented Jul 11, 2018

CLA assistant check
All committers have signed the CLA.

@webpack-bot
Copy link

@NaviMarella The most important CI builds failed. This way your PR can't be merged.

Please take a look at the CI results from travis (failure) and fix these issues.

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! I'll need to update the demo anyways soon!

@evenstensberg
Copy link
Member

Hi. The PR looks good, but you will need to update the commit to match the commit format we have. Look at the failing CI and other PRs. That should give you a good enough base for succeeding

@NaviMarella
Copy link
Contributor Author

I assume CI failed because of the commit message format, Also can you let me know what exactly has gone wrong as CI issues are not clear to understand.

@evenstensberg
Copy link
Member

Try using chore(docs): update xyz

@evenstensberg evenstensberg merged commit c3ab11b into webpack:master Jul 13, 2018
@evenstensberg
Copy link
Member

Thanks for submitting a PR! ❤️

dhruvdutt pushed a commit that referenced this pull request Jul 17, 2018
The link provided in [ webpack-scaffold-yourpackage ] section is missing. I have updated the link, Please verify and accept this PR.
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.

5 participants