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

[new release] opam-monorepo (0.2.1) #17965

Merged
merged 4 commits into from
Jan 15, 2021

Conversation

NathanReb
Copy link
Contributor

Assemble and manage fully vendored Dune repositories

CHANGES:

Fixed

CHANGES:

### Fixed

- Fix `--recurse-opam` option for the monorepo lock phase: correctly perform special directory
  filtering, add an error message when two versions of the same package opam file exist in the
  source tree, perform package name filtering before checking for uniqueness (tarides/opam-monorepo#151, @TheLortex)
@NathanReb
Copy link
Contributor Author

I also updated opam-monorepo.0.2.0 for which I forgot to add the depexts!

@camelus
Copy link
Contributor

camelus commented Jan 13, 2021

Commit: 3816dfe

A pull request by opam-seasoned @NathanReb.

☀️ All lint checks passed 3816dfe
  • These packages passed lint tests: opam-monorepo.0.2.0, opam-monorepo.0.2.1

☀️ Installability check (+1)
  • new installable packages (1): opam-monorepo.0.2.1

@NathanReb
Copy link
Contributor Author

I'll file an issue on opam-monorepo so we can improve the situation in the future but to clarify, we should make it so it always depends on conf-* packages when possible, only falling back to depexts when there is no such package.

All these dependencies would be in a separate virtual opam package.

Does that sound right to you?

@kit-ty-kate
Copy link
Member

yes that sounds perfect! Thanks a lot!

@kit-ty-kate
Copy link
Member

Thanks a lot!

@kit-ty-kate kit-ty-kate merged commit df43578 into ocaml:master Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants