summaryrefslogtreecommitdiff
path: root/crates/bot-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/bot-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/bot-http/src/github.rs')
-rw-r--r--crates/bot-http/src/github.rs35
1 files changed, 0 insertions, 35 deletions
diff --git a/crates/bot-http/src/github.rs b/crates/bot-http/src/github.rs
deleted file mode 100644
index 7822eb8..0000000
--- a/crates/bot-http/src/github.rs
+++ /dev/null
@@ -1,35 +0,0 @@
-use super::{ClientExt as _, Error};
-use crate::model::PullRequest;
-
-use std::future::Future;
-
-const GITHUB_API: &str = "https://api.github.com";
-
-pub trait ClientExt {
- /// Get the commit that merged [`pr`] in [`repo_owner`]/[`repo_name`]
- ///
- /// # Errors
- ///
- /// Will return [`Err`] if the merge commit cannot be found
- fn merge_commit_for(
- &self,
- repo_owner: &str,
- repo_name: &str,
- pr: u64,
- ) -> impl Future<Output = Result<Option<String>, Error>> + Send;
-}
-
-impl ClientExt for super::Client {
- async fn merge_commit_for(
- &self,
- repo_owner: &str,
- repo_name: &str,
- pr: u64,
- ) -> Result<Option<String>, Error> {
- let url = format!("{GITHUB_API}/repos/{repo_owner}/{repo_name}/pulls/{pr}");
- let resp: PullRequest = self.get_json(&url).await?;
- let merge_commit = resp.merge_commit_sha;
-
- Ok(merge_commit)
- }
-}