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

opam-dune-lint: update to latest version #337

Merged
merged 1 commit into from
Apr 30, 2021

Conversation

avsm
Copy link
Member

@avsm avsm commented Apr 28, 2021

This (I think) fixes some spurious failures I'm seeing with subpackages.

@talex5
Copy link
Contributor

talex5 commented Apr 28, 2021

Pushed to live-engine. Does your build work now if you rebuild it?

@avsm
Copy link
Member Author

avsm commented Apr 28, 2021

Hm no: https://ci.ocamllabs.io/github/mirage/ocaml-cf/commit/8d554fb9bca85dd32fdeb3f7f65aad95190bec00/variant/(lint-opam)

But oddly, running opam dune-lint with the released version works on my local checkout of the repo.

@talex5
Copy link
Contributor

talex5 commented Apr 30, 2021

Well, the upgrade seems fine anyway. Looks like a dune problem: ocurrent/opam-dune-lint#29

@talex5 talex5 merged commit f963fb3 into ocurrent:master Apr 30, 2021
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

Successfully merging this pull request may close these issues.

2 participants