diff options
| author | DanConwayDev <DanConwayDev@protonmail.com> | 2024-11-26 07:48:30 +0000 |
|---|---|---|
| committer | DanConwayDev <DanConwayDev@protonmail.com> | 2024-11-26 12:26:47 +0000 |
| commit | a0593e3aa9b19b9ca3c3881cbe0d9d207fe46d2c (patch) | |
| tree | 3482e7c558eff09cc91604ad6ddcd02cf2c94699 /src/lib/git_events.rs | |
| parent | 1332c625b75fce616e06b415c99a068bc45c8210 (diff) | |
refactor: err msgs 'cannot' > 'failed to'
in nearly all cases 'cannot' was used when an action was tried and
failed. 'failed to' is strictly better because:
* just because the action didn't work that time doesnt mean it
cannot work
* it is better at drawing the users attention to a problem
Diffstat (limited to 'src/lib/git_events.rs')
| -rw-r--r-- | src/lib/git_events.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/git_events.rs b/src/lib/git_events.rs index 29c4cf2..bfe5b30 100644 --- a/src/lib/git_events.rs +++ b/src/lib/git_events.rs | |||
| @@ -103,7 +103,7 @@ pub async fn generate_patch_event( | |||
| 103 | nostr::event::Kind::GitPatch, | 103 | nostr::event::Kind::GitPatch, |
| 104 | git_repo | 104 | git_repo |
| 105 | .make_patch_from_commit(commit, &series_count) | 105 | .make_patch_from_commit(commit, &series_count) |
| 106 | .context(format!("cannot make patch for commit {commit}"))?, | 106 | .context(format!("failed to make patch for commit {commit}"))?, |
| 107 | [ | 107 | [ |
| 108 | repo_ref | 108 | repo_ref |
| 109 | .maintainers | 109 | .maintainers |
| @@ -558,7 +558,7 @@ pub fn get_most_recent_patch_with_ancestors( | |||
| 558 | } | 558 | } |
| 559 | }) | 559 | }) |
| 560 | }) | 560 | }) |
| 561 | .context("cannot find patches_with_youngest_created_at")? | 561 | .context("failed to find patches_with_youngest_created_at")? |
| 562 | .id | 562 | .id |
| 563 | .to_string(); | 563 | .to_string(); |
| 564 | 564 | ||