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

Docs: Use GitHub Multiple PR/Issue templates #9911

Merged
merged 1 commit into from Feb 2, 2018
Merged

Conversation

kaicataldo
Copy link
Member

@kaicataldo kaicataldo commented Jan 30, 2018

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[X] Other, please explain:

This change utilizes GitHub's new multiple issue and pull request templates feature to make it so that users don't have to copy and paste the the template questions manually when they're filing a new issue.

What changes did you make? (Give an overview)
Added more issues templates for the various types of issues we see reported.

Is there anything you'd like reviewers to focus on?
Thoughts on this change?

Also, I think this would be helpful for PRs as well, though I'm not sure how we would do that. Any ideas?

Wasn't entirely sure what labels to put on this, so please feel free to update if you deem it necessary!

@kaicataldo kaicataldo added documentation Relates to ESLint's documentation infrastructure Relates to the tools used in the ESLint development process labels Jan 30, 2018
Copy link
Member

@gyandeeps gyandeeps left a comment

Choose a reason for hiding this comment

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

💯

@platinumazure
Copy link
Member

Question: Should we consolidate/move items in the templates folder into .github/ISSUE_TEMPLATE and update the website's issue guides to point directly there? I'm not sure if there's any reason to keep both directories but let me know if I'm missing something.

@kaicataldo
Copy link
Member Author

The only issue with that is that we currently reference those templates in the PR template. Ideally, I'd like to do the same thing for PRs (linking to various PR templates) but I wasn't sure how to link to that URL, since it seems like it would be different for each PR. But maybe I'm wrong about that?

If we can figure out how to do that, we can definitely remove the templates in the templates directory, which would be great!

@platinumazure
Copy link
Member

but I wasn't sure how to link to that URL, since it seems like it would be different for each PR

Not sure I understand this part? How would it be different for new PRs?

Copy link
Member

@platinumazure platinumazure 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! (My question is definitely not meant to be a blocker)

@kaicataldo
Copy link
Member Author

Not sure I understand this part? How would it be different for new PRs?

Please correct me if I'm wrong, but I believe PR urls are unique per PR and look like: https://github.com/eslint/eslint/compare/master...my-branch. Is there an easy way to append ?template=some-template.md to these unique URLs?

@platinumazure
Copy link
Member

platinumazure commented Feb 1, 2018

Please correct me if I'm wrong, but I believe PR urls are unique per PR and look like: https://github.com/eslint/eslint/compare/master...my-branch. Is there an easy way to append ?template=some-template.md to these unique URLs?

Good call, you're right-- if someone creates a pull request from a branch page or using the "recently pushed branches" div.

However, there is a "New Pull Request" button on the PR list which links to (e.g.) https://github.com/eslint/eslint/compare, which could easily have a template on it. That page lets you pick the base and head branch; however, I admit it might not be an intuitive interface (and might not work for users' forks at all). So I think we would need to dig a bit more if we wanted to implement this for PRs as well.

@not-an-aardvark
Copy link
Member

Personally, I almost never use the https://github.com/eslint/eslint/compare page to create a PR -- I look for the yellow bar on the front page of the repo that appears after I push a branch.

@kaicataldo
Copy link
Member Author

kaicataldo commented Feb 2, 2018

How do we feel about merging this now and looking into using multiple PR templates in the future? Seems like merging this now would be an immediate win for anyone making a new issue :)

@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Aug 3, 2018
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Aug 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived due to age This issue has been archived; please open a new issue for any further discussion documentation Relates to ESLint's documentation infrastructure Relates to the tools used in the ESLint development process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants