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

chore: bump dune version to 3.18 #11478

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

maiste
Copy link
Collaborator

@maiste maiste commented Feb 11, 2025

This PR is intended to be merged before #11275. The idea is to update the version of the dune lang first, as it implies new changes in terms of formatting. Then we can merge the PR that supports the maintenance fields on top of it.

cc @art-w

Signed-off-by: Etienne Marais <dev@maiste.fr>
Signed-off-by: Etienne Marais <dev@maiste.fr>
@maiste
Copy link
Collaborator Author

maiste commented Feb 12, 2025

I thought it would solve the problem, but I'm not convinced because of what the CI shows. As we need < 3.17 to have access to the formatting, it means we have to upgrade the dune-project to 3.18. However, as the version is not released yet, it will break all the local switch test as they are trying to pin 3.17.2. It is like an egg-and-chicken problem. Maybe we should wait for 3.18 to be released before doing this?

@maiste maiste added the chore Something that just needs to be done. label Feb 12, 2025
@Leonidas-from-XIV
Copy link
Collaborator

Yeah, I think that's (unfortunately) probably the best way to go about it. Let's revisit this after 3.18.

@maiste maiste marked this pull request as draft February 12, 2025 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Something that just needs to be done.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants