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 /test_utils/src/git.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 'test_utils/src/git.rs')
| -rw-r--r-- | test_utils/src/git.rs | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test_utils/src/git.rs b/test_utils/src/git.rs index 470cd83..2562799 100644 --- a/test_utils/src/git.rs +++ b/test_utils/src/git.rs | |||
| @@ -265,7 +265,7 @@ impl GitTestRepo { | |||
| 265 | let branch = self | 265 | let branch = self |
| 266 | .git_repo | 266 | .git_repo |
| 267 | .find_branch(branch_name, git2::BranchType::Local) | 267 | .find_branch(branch_name, git2::BranchType::Local) |
| 268 | .context(format!("cannot find branch {branch_name}"))?; | 268 | .context(format!("failed to find branch {branch_name}"))?; |
| 269 | Ok(branch.into_reference().peel_to_commit()?.id()) | 269 | Ok(branch.into_reference().peel_to_commit()?.id()) |
| 270 | } | 270 | } |
| 271 | 271 | ||