diff options
author | Sumner Evans <me@sumnerevans.com> | 2021-05-20 13:17:29 -0600 |
---|---|---|
committer | Alyssa Ross <hi@alyssa.is> | 2021-05-24 13:25:49 +0000 |
commit | d0b52edacec6a01b65b86b0e8bb5575470f1a775 (patch) | |
tree | 8576cf04f61c9bb0662e148d7a501fd03605065b /src/main.rs | |
parent | 4398773a589bf59df5fa4f9c6eff3d3f1c1ab441 (diff) | |
download | pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar.gz pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar.bz2 pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar.lz pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar.xz pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.tar.zst pr-tracker-d0b52edacec6a01b65b86b0e8bb5575470f1a775.zip |
Add title to PR information display
Also refactor MergeInfo -> PrInfo. Reviewed-by: Alyssa Ross <hi@alyssa.is> Tested-by: Alyssa Ross <hi@alyssa.is>
Diffstat (limited to 'src/main.rs')
-rw-r--r-- | src/main.rs | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/main.rs b/src/main.rs index 22023a8..0b70a15 100644 --- a/src/main.rs +++ b/src/main.rs @@ -1,5 +1,6 @@ // SPDX-License-Identifier: AGPL-3.0-or-later WITH GPL-3.0-linking-exception // SPDX-FileCopyrightText: 2021 Alyssa Ross <hi@alyssa.is> +// SPDX-FileCopyrightText: 2021 Sumner Evans <me@sumnerevans.com> mod branches; mod github; @@ -70,6 +71,7 @@ static GITHUB_TOKEN: Lazy<OsString> = Lazy::new(|| { struct PageTemplate { error: Option<String>, pr_number: Option<String>, + pr_title: Option<String>, closed: bool, tree: Option<Tree>, source_url: String, @@ -97,7 +99,7 @@ async fn track_pr(pr_number: Option<String>, status: &mut u16, page: &mut PageTe let github = GitHub::new(&GITHUB_TOKEN, &CONFIG.user_agent); - let merge_info = match github.merge_info_for_nixpkgs_pr(pr_number_i64).await { + let pr_info = match github.pr_info_for_nixpkgs_pr(pr_number_i64).await { Err(github::Error::NotFound) => { *status = 404; page.error = Some(format!("No such nixpkgs PR #{}.", pr_number_i64)); @@ -114,18 +116,19 @@ async fn track_pr(pr_number: Option<String>, status: &mut u16, page: &mut PageTe }; page.pr_number = Some(pr_number); + page.pr_title = Some(pr_info.title); - if matches!(merge_info.status, PullRequestStatus::Closed) { + if matches!(pr_info.status, PullRequestStatus::Closed) { page.closed = true; return; } let nixpkgs = Nixpkgs::new(&CONFIG.path, &CONFIG.remote); - let tree = Tree::make(merge_info.branch.to_string(), &merge_info.status, &nixpkgs).await; + let tree = Tree::make(pr_info.branch.to_string(), &pr_info.status, &nixpkgs).await; if let github::PullRequestStatus::Merged { merge_commit_oid, .. - } = merge_info.status + } = pr_info.status { if merge_commit_oid.is_none() { page.error = Some("For older PRs, GitHub doesn't tell us the merge commit, so we're unable to track this PR past being merged.".to_string()); |