mathieu/os-rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix changelog generation after repo move

Browse source
This commit is contained in:
Laurențiu Nicola 2022-04-19 09:07:10 +03:00
parent 65fbe0a8d1
commit d4dcb16a02
1 changed files with 56 additions and 46 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

102
xtask/src/release/changelog.rs
View file

@ -11,56 +11,53 @@ pub(crate) fn get_changelog(
prev_tag: &str, prev_tag: &str,
today: &str, today: &str,
) -> anyhow::Result<String> { ) -> anyhow::Result<String> {
let git_log = cmd!(sh, "git log {prev_tag}..HEAD --merges --reverse").read()?; let token = match env::var("GITHUB_TOKEN") {
Ok(token) => token,
Err(_) => bail!("Please obtain a personal access token from https://github.com/settings/tokens and set the `GITHUB_TOKEN` environment variable."),
};
let git_log = cmd!(sh, "git log {prev_tag}..HEAD --reverse").read()?;
let mut features = String::new(); let mut features = String::new();
let mut fixes = String::new(); let mut fixes = String::new();
let mut internal = String::new(); let mut internal = String::new();
let mut others = String::new(); let mut others = String::new();
for line in git_log.lines() { for line in git_log.lines() {
let line = line.trim_start(); let line = line.trim_start();
if let Some(p) = line.find(':') { if let Some(pr_num) = parse_pr_number(&line) {
let pr = &line[..p]; let accept = "Accept: application/vnd.github.v3+json";
if let Ok(pr_num) = pr.parse::<u32>() { let authorization = format!("Authorization: token {}", token);
let accept = "Accept: application/vnd.github.v3+json"; let pr_url = "https://api.github.com/repos/rust-lang/rust-analyzer/issues";
let token = match env::var("GITHUB_TOKEN") {
Ok(token) => token,
Err(_) => bail!("Please obtain a personal access token from https://github.com/settings/tokens and set the `GITHUB_TOKEN` environment variable."),
};
let authorization = format!("Authorization: token {}", token);
let pr_url = "https://api.github.com/repos/rust-analyzer/rust-analyzer/issues";
// we don't use an HTTPS client or JSON parser to keep the build times low // we don't use an HTTPS client or JSON parser to keep the build times low
let pr_json = let pr = pr_num.to_string();
cmd!(sh, "curl -s -H {accept} -H {authorization} {pr_url}/{pr}").read()?; let pr_json =
let pr_title = cmd!(sh, "jq .title").stdin(&pr_json).read()?; cmd!(sh, "curl -s -H {accept} -H {authorization} {pr_url}/{pr}").read()?;
let pr_title = unescape(&pr_title[1..pr_title.len() - 1]); let pr_title = cmd!(sh, "jq .title").stdin(&pr_json).read()?;
let pr_comment = cmd!(sh, "jq .body").stdin(pr_json).read()?; let pr_title = unescape(&pr_title[1..pr_title.len() - 1]);
let pr_comment = cmd!(sh, "jq .body").stdin(pr_json).read()?;
let comments_json = let comments_json =
cmd!(sh, "curl -s -H {accept} -H {authorization} {pr_url}/{pr}/comments") cmd!(sh, "curl -s -H {accept} -H {authorization} {pr_url}/{pr}/comments").read()?;
.read()?; let pr_comments = cmd!(sh, "jq .[].body").stdin(comments_json).read()?;
let pr_comments = cmd!(sh, "jq .[].body").stdin(comments_json).read()?;
let l = iter::once(pr_comment.as_str()) let l = iter::once(pr_comment.as_str())
.chain(pr_comments.lines()) .chain(pr_comments.lines())
.rev() .rev()
.find_map(|it| { .find_map(|it| {
let it = unescape(&it[1..it.len() - 1]); let it = unescape(&it[1..it.len() - 1]);
it.lines().find_map(parse_changelog_line) it.lines().find_map(parse_changelog_line)
}) })
.into_iter() .into_iter()
.next() .next()
.unwrap_or_else(|| parse_title_line(&pr_title)); .unwrap_or_else(|| parse_title_line(&pr_title));
let s = match l.kind { let s = match l.kind {
PrKind::Feature => &mut features, PrKind::Feature => &mut features,
PrKind::Fix => &mut fixes, PrKind::Fix => &mut fixes,
PrKind::Internal => &mut internal, PrKind::Internal => &mut internal,
PrKind::Other => &mut others, PrKind::Other => &mut others,
PrKind::Skip => continue, PrKind::Skip => continue,
}; };
writeln!(s, "* pr:{}[] {}", pr_num, l.message.as_deref().unwrap_or(&pr_title)) writeln!(s, "* pr:{}[] {}", pr_num, l.message.as_deref().unwrap_or(&pr_title)).unwrap();
.unwrap();
}
} }
} }
@ -73,11 +70,6 @@ pub(crate) fn get_changelog(
Commit: commit:{}[] + Commit: commit:{}[] +
Release: release:{}[] Release: release:{}[]
== Sponsors
**Become a sponsor:** On https://opencollective.com/rust-analyzer/[OpenCollective] or
https://github.com/sponsors/rust-analyzer[GitHub Sponsors].
== New Features == New Features
{} {}
@ -117,6 +109,24 @@ fn unescape(s: &str) -> String {
s.replace(r#"\""#, "").replace(r#"\n"#, "\n").replace(r#"\r"#, "") s.replace(r#"\""#, "").replace(r#"\n"#, "\n").replace(r#"\r"#, "")
} }
fn parse_pr_number(s: &str) -> Option<u32> {
const BORS_PREFIX: &str = "Merge #";
const HOMU_PREFIX: &str = "Auto merge of #";
if s.starts_with(BORS_PREFIX) {
let s = &s[BORS_PREFIX.len()..];
s.parse().ok()
} else if s.starts_with(HOMU_PREFIX) {
let s = &s[HOMU_PREFIX.len()..];
if let Some(space) = s.find(' ') {
s[..space].parse().ok()
} else {
None
}
} else {
None
}
}
fn parse_changelog_line(s: &str) -> Option<PrInfo> { fn parse_changelog_line(s: &str) -> Option<PrInfo> {
let parts = s.splitn(3, ' ').collect::<Vec<_>>(); let parts = s.splitn(3, ' ').collect::<Vec<_>>();
if parts.len() < 2 || parts[0] != "changelog" { if parts.len() < 2 || parts[0] != "changelog" {