Skip to content
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.

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

Check that the dune constraint matches the dune-project file #28

Closed
kit-ty-kate opened this issue Apr 6, 2021 · 1 comment
Closed
Assignees

Comments

@kit-ty-kate
Copy link
Contributor

Currently opam-dune-lint does not check for the validity of the dune constraint however this is a desirable feature that would make adding an opam-dune-lint phase to opam-repo-ci desirable in the future (we might want to think about the implications of having every explicit dependencies be listed, first)

Implemented in opam-repo-ci in ocurrent/opam-repo-ci#90. Feel free to copy/paste what's needed.

@moyodiallo
Copy link
Collaborator

This is fixed, thanks for the indication. Your implementation was very useful, with a slight modification it was OK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants