-
Notifications
You must be signed in to change notification settings - Fork 724
bump version numbers for 3.16.1.0 #11307
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
Conversation
Mikolaj
left a comment
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.
There are a few more changes in https://github.com/haskell/cabal/pull/11037/files (though I agree tweaking the boostrap files by hand is not likely to help).
|
Bootstrap is actually a problem: until 9.14.1 is actually released, we can't add it to the bootstrap versions. (A prerelease won't work for this, AIUI.) |
|
hot take: bootstrap isn't related to release directly and can be updated anytime afterwards? I'm not sure how people use it. Of course, someone can try to check out the 3.16.1.0 tag and bootstrap from there. But there's a general problem: I clearly remember us deciding that we only support bootstraping from the version on the master branch (we had to burn some bridges in the past, so that we don't have to maintain working bootstrapping across several branches, which is too much work). |
|
Now that you mention that, I recall that as well. So we can skip that part. |
|
Right, so bootstrap isn't applicable here and I think neither are the internal packages (bump with major version yes, with minors no; possibly |
ulysses4ever
left a comment
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.
Thanks! Just one question.
|
Actually, if we don't support bootstrapping from release branches anyway, it shouldn't be relevant. (If we did, and those bounds affected the bounds |
|
Oh, right, looking at this again I did it because it was done for 3.14.1.0 as well. |
|
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 06b79de |
|
I don't know why it didn't act on the label; it looks like all conditions match…. |
Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR: