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

Remove opam 2.1 support from the release script #6084

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

kit-ty-kate
Copy link
Member

@kit-ty-kate kit-ty-kate commented Jul 9, 2024

Simplifies the release script and makes it less likely to have errors
Related to #6082 and #6083

Backported to 2.2 in #6085

@kit-ty-kate kit-ty-kate added this to the 2.3.0~alpha milestone Jul 9, 2024
Copy link
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

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

Shouldn't we also add in the release readme the fact that now you need to be in the branch you want to release (and not just fetch the tag)

@kit-ty-kate
Copy link
Member Author

done

Copy link
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

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

Thanks!

@kit-ty-kate kit-ty-kate merged commit 5258a83 into ocaml:master Jul 10, 2024
2 checks passed
@kit-ty-kate kit-ty-kate deleted the rm-2.1-from-2.2-release branch July 10, 2024 13:43
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.

2 participants