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/bin/ngit/sub_commands/send.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/bin/ngit/sub_commands/send.rs')
| -rw-r--r-- | src/bin/ngit/sub_commands/send.rs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/bin/ngit/sub_commands/send.rs b/src/bin/ngit/sub_commands/send.rs index 114a021..1965875 100644 --- a/src/bin/ngit/sub_commands/send.rs +++ b/src/bin/ngit/sub_commands/send.rs | |||
| @@ -45,7 +45,7 @@ pub struct SubCommandArgs { | |||
| 45 | 45 | ||
| 46 | #[allow(clippy::too_many_lines)] | 46 | #[allow(clippy::too_many_lines)] |
| 47 | pub async fn launch(cli_args: &Cli, args: &SubCommandArgs, no_fetch: bool) -> Result<()> { | 47 | pub async fn launch(cli_args: &Cli, args: &SubCommandArgs, no_fetch: bool) -> Result<()> { |
| 48 | let git_repo = Repo::discover().context("cannot find a git repository")?; | 48 | let git_repo = Repo::discover().context("failed to find a git repository")?; |
| 49 | let git_repo_path = git_repo.get_path()?; | 49 | let git_repo_path = git_repo.get_path()?; |
| 50 | 50 | ||
| 51 | let (main_branch_name, main_tip) = git_repo | 51 | let (main_branch_name, main_tip) = git_repo |
| @@ -83,7 +83,7 @@ pub async fn launch(cli_args: &Cli, args: &SubCommandArgs, no_fetch: bool) -> Re | |||
| 83 | } else { | 83 | } else { |
| 84 | git_repo | 84 | git_repo |
| 85 | .parse_starting_commits(&args.since_or_range) | 85 | .parse_starting_commits(&args.since_or_range) |
| 86 | .context("cannot parse specified starting commit or range")? | 86 | .context("failed to parse specified starting commit or range")? |
| 87 | } | 87 | } |
| 88 | }; | 88 | }; |
| 89 | 89 | ||