From bbc00edc6508ea1910c4d9e6a272f7859900950d Mon Sep 17 00:00:00 2001
From: seth <getchoo@tuta.io>
Date: Fri, 16 Aug 2024 22:14:16 -0400
Subject: 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
---
 crates/bot-http/src/github.rs | 35 -----------------------------------
 1 file changed, 35 deletions(-)
 delete mode 100644 crates/bot-http/src/github.rs

(limited to 'crates/bot-http/src/github.rs')

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)
-	}
-}
-- 
cgit v1.2.3