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] routes (0.6.0) #15560

Closed
wants to merge 1 commit into from

Conversation

anuragsoni
Copy link
Contributor

CHANGES:

* Improve mdx test tules (anuragsoni/routes#73, @NathanReb)
* Use github actions instead of travis ci (anuragsoni/routes#70, anuragsoni/routes#72)
* Get human readable route pattern from a route (anuragsoni/routes#64, anuragsoni/routes#74)
* Allow writing custom path match patterns (anuragsoni/routes#76)
]
depends: [
"ocaml" {>= "4.05"}
"dune" { >= "1.1" }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure that (using fmt 1.1) works with dune 1.1 and does not require 1.5?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so. My understanding was that the version needs to match the dune language version in the dune-project file. I’ll check the docs again to confirm

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently dune extensions do not check for a matching dune version number (see ocaml/dune#2957) and according to ocaml/dune#1821, fmt 1.1 has been added in dune.1.7.0. So I believe your package requires at least dune 1.7

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for sharing those links. I think if I need to change the dune version constraint, I’ll go over the dune docs to see if there are any other extension that I’d like to use. Or see if I stop using the fmt extension. I’ll close this pull request for now and revisit this once I’ve figured out which dune extensions I need.

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