-
Notifications
You must be signed in to change notification settings - Fork 410
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
Bump Dune lang version to 3.18 #11175
Conversation
Signed-off-by: Nicolás Ojeda Bär <[email protected]>
Signed-off-by: Nicolás Ojeda Bär <[email protected]>
CI failures look unrelated. |
I've retriggered the failing tests manually and some failures are related (albeit easily fixed). |
Signed-off-by: Nicolás Ojeda Bär <[email protected]>
Thanks, fixed. |
doc/test/run.t
Outdated
../melange.rst: (lang dune 3.17) | ||
../sites.rst: (lang dune 3.17) | ||
../sites.rst: (lang dune 3.17) | ||
../sites.rst: (lang dune 3.17) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These probably need to be updated, as written in the description above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, done.
Signed-off-by: Nicolás Ojeda Bär <[email protected]>
This reverts commit ed9c163. Signed-off-by: Nicolás Ojeda Bär <[email protected]>
e6abd81
to
1f63f3a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks! The remaining failures are unrelated.
See #11166 (comment)
The second commit fixes the versioning guard of a change that was initially in 3.17 but was later reverted (see #11164).