New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve cli abstractions #2

Merged

filipeom merged 1 commit into main from improve-cli-abstractions

Sep 14, 2024

Commits on Sep 12, 2024

Improve cli abstractions

Use `Cmdliner.Arg.file` conv to parse valid files. Cmdliners' file
conv already does a `Sys.file_exists` so we avoid code duplication
here.

Also define `Mode.t` as an algebraic datatype to reduce string
allocations and allow for faster instant `is_` mode checks.

filipeom committed Sep 12, 2024