rei-mirrors
/
Fornjot
mirror of https://github.com/hannobraun/Fornjot
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix release announcements including older PRs 

Previously, pull requests merged before the most recent release were
still included, if they were updated after the most recent release.
This commit is contained in:
Hanno Braun 2022-11-08 11:19:39 +01:00
parent 3922d3d537
commit 7a04c11d17
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
tools/automator/src/pull_requests.rs
View File

@ -20,7 +20,7 @@ impl PullRequestsSinceLastRelease {
let mut pull_requests = BTreeMap::new();
let mut page = 1u32;
let version_of_last_release = 'outer: loop {
let (version_of_last_release, time_of_last_release) = 'outer: loop {
const MAX_RESULTS_PER_PAGE: u8 = 100;
println!("Fetching page {}...", page);
@ -60,7 +60,12 @@ impl PullRequestsSinceLastRelease {
)
})?;
break 'outer version;
let time =
pull_request.merged_at.ok_or_else(|| {
anyhow!("Release PR is missing merge time")
})?;
break 'outer (version, time);
}
}
}
@ -99,6 +104,10 @@ impl PullRequestsSinceLastRelease {
}
};
pull_requests.retain(|_, pull_request| {
pull_request.merged_at > time_of_last_release
});
Ok(Self {
pull_requests,
version_of_last_release,