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
Update PR template to be simpler and mention allowing edits #4775
Conversation
SgtCoDFish
commented
Jan 24, 2022
•
edited
edited
- Removes link to k8s contributor guidelines; they're way more complicated than what we require. We don't want to link to docs suggesting the user needs to sign the k8s CLA.
- Removes link to k8s "faster reviews" - again, waaaay more complicated than we need. We don't want to link to docs suggesting "File a Kubernetes Enhancement Proposal"
- Adds link to our contributor guidelines.
- Adds special note about DCO signoff since it's often something that people get confused about
- Adds a link explaining how to allow edits by maintainers, inspired by the DocSearch PR instructions.
- Removes a couple of headings that nobody ever followed; instead have simpler clearer headings
- Default to release-note-none. Maintainers can always edit PR descriptions to add a release note even if the PR submitter didn't. This way, the default case is mergeable and it's one less thing for people to have to worry about.
- Add a prepopulated section suggesting which "kind" labels are available and prompting the submitter to pick one
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
suggestion: add a /kind section with a corresponding brief explanation. I personally always forget it.
Ironically, this PR needs a kind.
Nice, thank you, Ashley, that's something I wished to do but never got around doing it! 🙏 /kind cleanup |
Ouch, this is not a PR that should require any test to be run, but go figure :( /retest |
That makes the case for adding it very well, doesn't it? I'll add it now /hold |
I've added a section for kinds, but I'm not really thrilled with it. Feedback appreciated. /unhold |
this can't include actaul examples because they'd be picked up by prow Signed-off-by: Ashley Davis <ashley.davis@jetstack.io>
7a7d536
to
b0f8a6e
Compare
/retest |
19 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
This is a great improvement, thanks! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jahrlin, jakexks, SgtCoDFish The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
1 similar comment
/retest |
/test pull-cert-manager-e2e-v1-23 |
/retest |