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

misc: reduce hop for creating an issue #718

Merged
merged 2 commits into from Dec 29, 2018

Conversation

hemal7735
Copy link
Member

@hemal7735 hemal7735 commented Dec 28, 2018

What kind of change does this PR introduce?
This is more of a feasibility kind of change for the benefit of Issue Raiser. Please feel free to close this PR if it adds a noise.

Currently, when we face any issue while running transformation we throw an error giving an URL
https://github.com/webpack/webpack-cli/issues

However, the bug reporter has to do one more hop to create an issue.

This PR changes that URL to https://github.com/webpack/webpack-cli/issues/new?template=Bug_report.md

Did you add tests for your changes? NA

If relevant, did you update the documentation? NA

Does this PR introduce a breaking change? NO

@hemal7735 hemal7735 changed the title misc: reduce 1 hop for creating an issue misc: reduce hop for creating an issue Dec 28, 2018
Copy link
Member

@rishabh3112 rishabh3112 left a comment

Choose a reason for hiding this comment

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

Just one change, rest LGTM 👍

packages/utils/modify-config-helper.ts Outdated Show resolved Hide resolved
packages/utils/modify-config-helper.ts Outdated Show resolved Hide resolved
@webpack-bot
Copy link

@rishabh3112 Thanks for your update.

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

@evenstensberg Please review the new changes.

Co-Authored-By: hemal7735 <hemal.patel7735@outlook.com>
@rishabh3112
Copy link
Member

rishabh3112 commented Dec 28, 2018

Got it.

@webpack-bot
Copy link

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

@hemal7735
Copy link
Member Author

Who pushed commit using my account 🤔

It is a GitHub co-commit feature, where suggestions can be directly committed, and it will show both names. In this case yours (suggested changes) and mine (accepted changes)

@rishabh3112
Copy link
Member

Well changed my comment accordingly.
I am on mobile right now which don't show co commiters 😅

@ematipico ematipico merged commit eb06af8 into webpack:master Dec 29, 2018
@hemal7735 hemal7735 deleted the better-issue-location branch December 29, 2018 05:01
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.

None yet

6 participants