-
Notifications
You must be signed in to change notification settings - Fork 148
Add template for Pull Requests #400
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
Comments
This one can be interesting too 👀 |
Yes! I asked beta access to it but still nothing. I'll enable it as soon as we have it. |
The Issue Forms feature is available for all public repositories, so you don't have to wait for anything. 😉 The new Forms feature is not available for pull requests, sadly.... I can make a bug report Issue Form and create a basic pull request template, if you want. Questions about bug report form
Questions about pull request template
|
You are right, I confused myself with the new issues board 🤦♂️ . The issue form is available for sure since we used it in Octoclairvoyant. In order to report bugs (and also for new feature issue form), the idea would be to have something similar to ESLint one but simplified. Let me answer those questions! What information must a bug reporter provide (operating system, npm version, node version, things like that) From the user environment, we need:
Do we need a field for error messages/log output, things like that? Yes, ideally the error output or a screenshot in case it's a false positive or something like that. Here would be nice to indicate what the user expected to happen and what actually happens. What input fields are required for a complete bug report? We would need to know:
What input fields are optional when filling in a bug report? Anything else I need to know before I can start?
About PR template: Specify what the guidelines are to fill in details about the PR? Flag and track necessary checks are addressed (which checks, specify!)
Do we want to remind contributor to use Fixes #1 to close the linked issue automatically? Anything else I need to know before I can start? I think that's all really Thank you very much @HonkingGoose! |
I think we still want forms for:
@Belco90 Can you explain the requirements for these forms? |
I've moved those remaining issue forms to #430 so we can close this one. Thanks @HonkingGoose! |
Add template for Pull Requests so:
Necessary checkbox to track necessary checks are:
The text was updated successfully, but these errors were encountered: