summaryrefslogtreecommitdiff
path: root/crates/nixpkgs-tracker-http/src/github.rs
diff options
context:
space:
mode:
authorseth <[email protected]>2024-08-16 22:14:16 -0400
committerGitHub <[email protected]>2024-08-17 02:14:16 +0000
commitbbc00edc6508ea1910c4d9e6a272f7859900950d (patch)
tree538fe23940cb6bca8afa48059e502bfa7d5608cd /crates/nixpkgs-tracker-http/src/github.rs
parent66ea6f8c225cc2273cfbfb1b948604c170a78e8c (diff)
end of summer refactor (#31)
* crates: `bot-*` -> `discord-bot` I didn't really need all these crates to be split :/ * discord-bot: revamp http impl also handles the new errors reported by teawieAPI * crates: split http backend this can be reused easily * git-tracker: short-circuit boolean logic We don't need to check if the commit is a descendant of the HEAD of the branch if it *is* the HEAD * nix: fenix -> nixpkgs * treefmt: add actionlint * nix: use docker arch names for containers * ci: use actions-rust-lang actions * nix: drop ci dev shell * git-tracker: init ManagedRepository this logic can be shared * ci: use nix for clippy scan * discord-bot: better handle unmerged PRs * ci: fix treefmt check * nix: fix clippy check * .env.template: update crate names * git-tracker: use remote name for remote name i was half asleep * discord-bot: handle merged PRs that aren't found in any tracked branches * git-tracker: make collect_statuses_in() return a Vec * discord-bot: add more PR info with response fixes #18
Diffstat (limited to 'crates/nixpkgs-tracker-http/src/github.rs')
-rw-r--r--crates/nixpkgs-tracker-http/src/github.rs40
1 files changed, 40 insertions, 0 deletions
diff --git a/crates/nixpkgs-tracker-http/src/github.rs b/crates/nixpkgs-tracker-http/src/github.rs
new file mode 100644
index 0000000..12ee832
--- /dev/null
+++ b/crates/nixpkgs-tracker-http/src/github.rs
@@ -0,0 +1,40 @@
+use super::{Error, PullRequest};
+
+use std::future::Future;
+
+use log::trace;
+
+const GITHUB_API: &str = "https://api.github.com";
+
+pub trait Ext {
+ /// GET `/repos/{repo_owner}/{repo_name}/pulls/{id}`
+ ///
+ /// # Errors
+ ///
+ /// Will return [`Err`] if the merge commit cannot be found
+ fn pull_request(
+ &self,
+ repo_owner: &str,
+ repo_name: &str,
+ id: u64,
+ ) -> impl Future<Output = Result<PullRequest, Error>> + Send;
+}
+
+impl Ext for super::Client {
+ async fn pull_request(
+ &self,
+ repo_owner: &str,
+ repo_name: &str,
+ id: u64,
+ ) -> Result<PullRequest, Error> {
+ let url = format!("{GITHUB_API}/repos/{repo_owner}/{repo_name}/pulls/{id}");
+
+ let request = self.get(&url).build()?;
+ trace!("Making GET request to `{}`", request.url());
+ let response = self.execute(request).await?;
+ response.error_for_status_ref()?;
+ let pull_request: PullRequest = response.json().await?;
+
+ Ok(pull_request)
+ }
+}