Skip to content

Commit

Permalink
Merge pull request #400 from mnxn/fix-string-interpolation
Browse files Browse the repository at this point in the history
Fix incomplete error message for missing commands
  • Loading branch information
mnxn authored Oct 9, 2020
2 parents 2697a0c + d92f21c commit 81fd8bc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Change Log

## Unreleased

- Fix incomplete error message for missing commands (#400)

## 1.3.1

- Fix bug that prevented the extension from initializing properly when installed
Expand Down
3 changes: 2 additions & 1 deletion src/cmd.ml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ let check_spawn { bin; args } =
| Some path -> (
let open Promise.Syntax in
which path bin >>| function
| None -> Error {j| Command "$bin" not found |j}
| None ->
Error (Printf.sprintf "Command %s not found" (Path.to_string bin))
| Some bin -> Ok { bin; args } )

let check t =
Expand Down

0 comments on commit 81fd8bc

Please sign in to comment.