Mathieu Barbin
Mathieu Barbin
> I suppose having a separate command would be a bit nicer OK. I'll see what I can do. I think I can squeeze an attempt into my dev schedule...
Work in progress in #6197
#6197 was merged and is planned for a future release. Do you wish to keep this issue open until the release, or may I close it now? Thank you!
https://ocaml.ci.dev/github/mbarbin/vcs seems to be getting this error pretty consistently. Job: `EXPERIMENTAL: (lint-opam) (failure: Failed: Build failed)` Error: `[ERROR] No package named deps_for_opam_dune_lint found`
Note to myself if we end up with this version: I'd like to propose to tweak it just a bit further: - for consistency within opam, consider keeping the same...
@kit-ty-kate I discovered the `OpamArg.mk_*` util, along with the *validity* of flags, and tried to use that. I think the code is missing the upcoming version `2.4` if I am...
> Yes you're correct, i forgot to add it when i bumped the version to 2.4. You can add it by simply adding the missing `OpamArg{,Tools}.cli2_4`, this is a simple...
Thank you @kit-ty-kate for the review and addressing these review comments!
@rjbou Thanks for looking into the PR! Re: Global options. My impression was that some might be applicable. Consider for example: ```sh --cli=MAJOR.MINOR (absent=2.4) Use the command-line interface syntax and...
> > Is `--cli` controlled by the global options? If so, I don't understand the purpose of setting the cli version in the code, if the flag cannot be used...