Skip to content
This repository has been archived by the owner on Oct 3, 2024. It is now read-only.

Commit

Permalink
Fail hard with a todo for missing opam repo
Browse files Browse the repository at this point in the history
Fits with the docs of the opam `local_opam_repo_term`, which says this
is optional, and that we will default in its absence.
  • Loading branch information
shonfeder committed Aug 21, 2024
1 parent 7c0fa3d commit 70af3d9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bin/main.ml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ let to_exit_code : (unit, string) result Term.t -> Cmd.Exit.code Term.t =

let lint (changed_pkgs, new_pkgs) local_repo_dir =
match local_repo_dir with
| None -> Error "No opam repository directory specified."
| None -> failwith "TODO: default to using the opam repository"
| Some d -> (
print_endline @@ Printf.sprintf "Linting opam-repository at %s ..." d;
match Lint.check ~new_pkgs ~changed_pkgs d with
Expand Down

0 comments on commit 70af3d9

Please sign in to comment.