From d25129d829e0ebd70b4e60e399fe91c0d80aa1ad Mon Sep 17 00:00:00 2001
From: seth <getchoo@tuta.io>
Date: Sun, 16 Jun 2024 07:15:13 -0400
Subject: use libgit2 to track PRs (#10)

* nix: don't depend on registry for nixpkgs input

* use libgit2 to track PRs

* nix: don't use ci devShell as defaul

* crates: bump serenity from `9ad74d4` to `0.12.2

* nix: fix cross compiled builds

* crates: split more from client

* bot-jobs: update remote refs more efficiently

* git-tracker: account for HEAD commits

* bot-config: use nixpkgs branches from environment

* bot-commands: don't display branches prs haven't landed in

* git-tracker: return false when commits aren't found

this is annoying as a hard error since it turns out github will report
garbage merge commit SHAs for PRs that *haven't* been merged yet. yay

* bot: improve docs in some places

* bot-client: display invite link on start

* bot-http: add TeawieClientExt

* bot-commands: add /about

* docs: update readme todos

* nix: enable StateDirectory in module

* crates: bump to 0.2.0
---
 src/http/github.rs | 69 ------------------------------------------------------
 1 file changed, 69 deletions(-)
 delete mode 100644 src/http/github.rs

(limited to 'src/http/github.rs')

diff --git a/src/http/github.rs b/src/http/github.rs
deleted file mode 100644
index 8d4f18a..0000000
--- a/src/http/github.rs
+++ /dev/null
@@ -1,69 +0,0 @@
-use super::{Error, HttpClientExt};
-
-use serde::Deserialize;
-
-pub const GITHUB_URL: &str = "https://github.com";
-pub const GITHUB_API: &str = "https://api.github.com";
-
-/// Bad version of `/repos/{owner}/{repo}/{compare}/{ref}...{ref}`
-#[derive(Deserialize)]
-struct Compare {
-	status: String,
-	ahead_by: i32,
-}
-
-/// Bad version of `/repos/{owner}/{repo}/pulls/{pull_number}`
-#[derive(Deserialize)]
-struct PullRequest {
-	merge_commit_sha: String,
-}
-
-pub trait GithubClientExt {
-	/// Get the commit that merged [`pr`] in [`repo_owner`]/[`repo_name`]
-	async fn merge_commit_for(
-		&self,
-		repo_owner: &str,
-		repo_name: &str,
-		pr: u64,
-	) -> Result<String, Error>;
-
-	/// Check if [`commit`] is in [`branch`] of [`repo_owner`]/[`repo_name`]
-	async fn is_commit_in_branch(
-		&self,
-		repo_owner: &str,
-		repo_name: &str,
-		branch_name: &str,
-		commit: &str,
-	) -> Result<bool, Error>;
-}
-
-impl GithubClientExt for super::Client {
-	async fn merge_commit_for(
-		&self,
-		repo_owner: &str,
-		repo_name: &str,
-		pr: u64,
-	) -> Result<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)
-	}
-
-	async fn is_commit_in_branch(
-		&self,
-		repo_owner: &str,
-		repo_name: &str,
-		branch: &str,
-		commit: &str,
-	) -> Result<bool, Error> {
-		let url = format!(
-			"https://api.github.com/repos/{repo_owner}/{repo_name}/compare/{branch}...{commit}"
-		);
-		let resp: Compare = self.get_json(&url).await?;
-		let in_branch = resp.status != "diverged" && resp.ahead_by >= 0;
-
-		Ok(in_branch)
-	}
-}
-- 
cgit v1.2.3