Skip to content

Commit c35ff9f

Browse files
authored
Merge pull request #1837 from Mark-Simulacrum/relnotes-issues
Enhancements to relnotes process
2 parents 330e790 + 53474af commit c35ff9f

File tree

1 file changed

+18
-8
lines changed

1 file changed

+18
-8
lines changed

src/handlers/relnotes.rs

+18-8
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,9 @@ pub async fn handle(ctx: &Context, event: &Event) -> anyhow::Result<()> {
6464
}
6565

6666
if let IssuesAction::Labeled { label } = &e.action {
67-
if label.name == "relnotes" || label.name == "relnotes-perf" {
67+
if ["relnotes", "relnotes-perf", "finished-final-comment-period"]
68+
.contains(&label.name.as_str())
69+
{
6870
let title = format!(
6971
"Tracking issue for release notes of #{}: {}",
7072
e.issue.number, e.issue.title
@@ -73,36 +75,44 @@ pub async fn handle(ctx: &Context, event: &Event) -> anyhow::Result<()> {
7375
"
7476
This issue tracks the release notes text for #{}.
7577
76-
- [ ] Issue is nominated for the responsible team (and T-release nomination is removed).
77-
- [ ] Proposed text is drafted by team responsible for underlying change.
78+
- [ ] Proposed text is drafted by PR author (or team) making the noteworthy change.
7879
- [ ] Issue is nominated for release team review of clarity for wider audience.
7980
- [ ] Release team includes text in release notes/blog posts.
8081
8182
Release notes text:
8283
83-
The section title will be de-duplicated by the release team with other release notes issues.
84-
Prefer to use the standard titles from [previous releases](https://doc.rust-lang.org/nightly/releases.html).
85-
More than one section can be included if needed.
84+
The responsible team for the underlying change should edit this section to replace the
85+
automatically generated link with a succinct description of what changed, drawing upon text
86+
proposed by the author (either in discussion or through direct editing). If the change is notable
87+
enough for inclusion in the blog post, add content to the blog section below.
8688
8789
```markdown
88-
# Compatibility Notes
90+
# Pick a category...
8991
- [{}]({})
9092
```
9193
94+
Note: The section title will be de-duplicated by the release team with other release notes issues.
95+
Please use a standard title from [previous releases](https://doc.rust-lang.org/nightly/releases.html).
96+
More than one section can be included if needed.
97+
9298
Release blog section (if any, leave blank if no section is expected):
9399
94100
```markdown
95101
```
102+
103+
cc {} -- origin issue/PR authors and assignees for starting to draft text
96104
",
97105
e.issue.number, e.issue.title, e.issue.html_url,
106+
[&e.issue.user].into_iter().chain(e.issue.assignees.iter())
107+
.map(|v| format!("@{}", v.login)).collect::<Vec<_>>().join(", ")
98108
);
99109
let resp = ctx
100110
.github
101111
.new_issue(
102112
&e.issue.repository(),
103113
&title,
104114
&body,
105-
vec!["relnotes".to_owned(), "I-release-nominated".to_owned()],
115+
vec!["relnotes".to_owned(), "relnotes-tracking-issue".to_owned()],
106116
)
107117
.await?;
108118
state.data.relnotes_issue = Some(resp.number);

0 commit comments

Comments
 (0)