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

[3.17] backport 11204 - remove errors message from dune subst with empty dir #11216

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

maiste
Copy link
Collaborator

@maiste maiste commented Dec 16, 2024

Backport for the 3.17.1 release.

@maiste maiste added the release PR or issue for releasing a new dune version label Dec 16, 2024
@maiste maiste requested a review from nojb December 16, 2024 13:37
@maiste maiste mentioned this pull request Dec 16, 2024
11 tasks
Copy link
Collaborator

@nojb nojb left a comment

Choose a reason for hiding this comment

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

LGTM

@nojb
Copy link
Collaborator

nojb commented Dec 16, 2024

There is a CI error, but looks unrelated.

Remove all error checking for empty projects in [$ dune subst]

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Signed-off-by: Etienne Marais <dev@maiste.fr>
@maiste
Copy link
Collaborator Author

maiste commented Dec 17, 2024

Your backport seemed to pass the CI. I have rebased, and we will see if the error is still here. If it does pass, I will merge this PR and prepare the release.

EDIT: the only part that does not succeed is the coverage. I merge it.

@maiste maiste merged commit 324e1b5 into ocaml:3.17 Dec 17, 2024
28 of 29 checks passed
@maiste maiste deleted the backport-11204 branch December 17, 2024 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release PR or issue for releasing a new dune version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants