-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Adds label prompt and explanation to pull request template #69837
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,21 +1,37 @@ | ||
<!-- Thanks for sending a pull request! Here are some tips for you: | ||
1. If this is your first time, read our contributor guidelines https://git.k8s.io/community/contributors/guide#your-first-contribution and developer guide https://git.k8s.io/community/contributors/devel/development.md#development-guide | ||
2. If you want *faster* PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews | ||
3. Follow the instructions for writing a release note: https://git.k8s.io/community/contributors/guide/release-notes.md | ||
4. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests | ||
2. Please label this pull request according to what type of issue you are addressing, especially if this is a release targeted pull request. For reference on required PR/issue labels, read here: | ||
https://git.k8s.io/community/contributors/devel/release.md#issue-kind-label | ||
3. If you want *faster* PR reviews, read how: https://git.k8s.io/community/contributors/guide/pull-requests.md#best-practices-for-faster-reviews | ||
4. Follow the instructions for writing a release note: https://git.k8s.io/community/contributors/guide/release-notes.md | ||
5. If the PR is unfinished, see how to mark it: https://git.k8s.io/community/contributors/guide/pull-requests.md#marking-unfinished-pull-requests | ||
--> | ||
|
||
**What type of PR is this?** | ||
> Uncomment only one, leave it on its own line: | ||
> | ||
> /kind api-change | ||
> /kind bug | ||
> /kind cleanup | ||
> /kind design | ||
> /kind documentation | ||
> /kind failing-test | ||
> /kind feature | ||
> /kind flake | ||
|
||
**What this PR does / why we need it**: | ||
|
||
**Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: | ||
Fixes # | ||
|
||
**Special notes for your reviewer**: | ||
|
||
**Release note**: | ||
<!-- Write your release note: | ||
1. Enter your extended release note in the below block. If the PR requires additional action from users switching to the new release, include the string "action required". | ||
2. If no release note is required, just write "NONE". | ||
**Does this PR introduce a user-facing change?**: | ||
<!-- | ||
If no, just write "NONE". | ||
If yes, a release note is required: | ||
Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required". | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @guineveresaenger -- We use the following language in https://git.k8s.io/contributors/guide/release-notes.md:
Does There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Given @cjwagner's feedback on Slack, looks like the relnote only needs to contain the phrase and the location shouldn't matter, so I'm happy now :) |
||
2. | ||
--> | ||
```release-note | ||
|
||
|
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.
what about /kind cleanup?
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.
Yes, I was wondering if it was perhaps too laborious for folks to choose from a long list of labels (esp since the issue template doesn't list them) but I'm happy to add them all.