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

Only enable package in short-form rules on dune lang 3.8+ #7476

Merged
merged 3 commits into from
Apr 11, 2023

Conversation

Leonidas-from-XIV
Copy link
Collaborator

Closes #7471, as discussed in #7461 to only allow the #7445 feature on dune-lang 3.8.

This fixes the issue that #7445 by itself fixes a language quirk that is only fixed in 3.8, but without actually requiring dune-lang 3.8. It revises the previous test that uses an older dune-lang to fail.

@rgrinberg rgrinberg added this to the 3.8.0 milestone Apr 3, 2023
Leonidas-from-XIV and others added 3 commits April 11, 2023 11:40
Signed-off-by: Etienne Millon <[email protected]>
Copy link
Collaborator

@emillon emillon left a comment

Choose a reason for hiding this comment

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

Thanks @Leonidas-from-XIV . I refactored the implementation. so that the atom is not duplicated into the check. Does that this right to you?

@Leonidas-from-XIV
Copy link
Collaborator Author

Ah yes, that's a much nicer implementation indeed!

@emillon emillon merged commit 4fa0bfa into ocaml:main Apr 11, 2023
@Leonidas-from-XIV Leonidas-from-XIV deleted the versioned-package-support branch April 12, 2023 08:30
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.

#7445 check needs to be versioned
3 participants