Minor formatting changes

Prom3th3us · May 6, 2024 · d53999e · d53999e
1 parent 940bc94
commit d53999e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/backend/src/bounty/api/claim.rs b/backend/src/bounty/api/claim.rs
@@ -75,6 +75,8 @@ pub async fn claim_impl(
                 github_pr_id: github_pr_id.clone(),
             }),
             Some(pull_request) => {
+                // TODO: unify GitHub errors
+                // FIXME: remove unwraps
                 let pr_response: Result<PrDetailsResponse, MergeDetailsErr> = github_client
                     .get_merged_details(extract_pull_number(&github_pr_id).unwrap())
                     .await;

diff --git a/backend/src/bounty/api/deposit.rs b/backend/src/bounty/api/deposit.rs
@@ -16,7 +16,7 @@ pub enum DepositErr {
 pub type DepositReceipt = Result<Tokens, DepositErr>;
 
 pub async fn deposit_impl() -> DepositReceipt {
-    // FIXME check caller equals the owner who initialized the bounty.
+    // FIXME: check caller equals the owner who initialized the bounty.
     let caller = caller();
     let icrc1_token_canister_id = Principal::from_text(MAINNET_ICRC1_LEDGER_CANISTER_ID)
         .expect("Wrong MAINNET_ICRC1_LEDGER_CANISTER_ID");
@@ -59,7 +59,7 @@ async fn deposit_icrc1(
         },
         amount: available.clone(),
         fee: Some(icrc1_token_fee),
-        // TODO enhance memo text
+        // TODO: enhance memo text
         memo: Some(Memo(0)),
         created_at_time: None,
     };

diff --git a/backend/src/provider/github/api/get_fixed_by.rs b/backend/src/provider/github/api/get_fixed_by.rs
@@ -90,6 +90,7 @@ fn remove_github_prefix(url: &str) -> String {
     url.replace("https://github.com/", "")
 }
 
+// TODO: use extract_regex
 fn extract_pull_request(html: &str) -> Option<String> {
     // Define a regular expression pattern to match the href attribute
     let re = Regex::new(r#"<a\s+[^>]*?href="(.*?)"[^>]*?>"#).unwrap();