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

Package yuujinchou.1.0.0 #20453

Merged
merged 1 commit into from
Jan 15, 2022
Merged

Package yuujinchou.1.0.0 #20453

merged 1 commit into from
Jan 15, 2022

Conversation

favonia
Copy link
Contributor

@favonia favonia commented Jan 14, 2022

yuujinchou.1.0.0

Name pattern combinators
This package implements a pattern language for selecting names. It intends to facilitate the implementation of the "import" statement or any feature that allows users to select a group of names by patterns.



🐫 Pull-request generated by opam-publish v2.1.0

@favonia
Copy link
Contributor Author

favonia commented Jan 14, 2022

I am a bit confused by the linter. The minimum version of dune is already 2.7. It seems the only violation, if any, is that I used pinned instead of dev. However, the documentation at https://dune.readthedocs.io/en/stable/opam.html indicates that I should use pinned. Any suggestions?

@kit-ty-kate
Copy link
Member

kit-ty-kate commented Jan 15, 2022

Sadly the online dune documentation isn’t up-to-date. See what the documentation is supposed to be generated from: https://github.com/ocaml/dune/blob/main/doc/opam.rst

The comment in the CI is at destinations of people who use dune’s autogeneration that was buggy until 2.7. However this isn’t your case so you can simply change it by hand.

See ocaml/dune#3647 for the full story if you’re curious

@favonia
Copy link
Contributor Author

favonia commented Jan 15, 2022

@kit-ty-kate Thank you. Every test is passed except the ones for macos-homebrew (experimental).

@mseri
Copy link
Member

mseri commented Jan 15, 2022

Thanks!

@mseri mseri merged commit b7abb37 into ocaml:master Jan 15, 2022
@favonia favonia deleted the opam-publish-yuujinchou.1.0.0 branch January 16, 2022 02:04
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.

3 participants