-
Notifications
You must be signed in to change notification settings - Fork 710
Add release notes for 3.14.2.0 release #10894
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
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.
LGTM, though we should also fill the C.2.8 “Unresolved” section, though I hope it will contain only ~2 items total (I'm not counting tickets that only concern problems working with very old Cabal versions, etc.).
Let's not merge before C1 is complete.
5e85109
to
80967c1
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.
Thank you.
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!
Below is a light copy-editing pass.
Also, as I mentioned in the comment above, I'd prefer unresolved issues to go in the announcement. Counterpoint: it's good to have a centralized record of these things, which means having them in the repo. And there's currently no better place than release-notes
, perhaps.
@mpickering: would you like to answer the comment and set the merge label? I think we can keep the 'Unresolved" section where it is. |
I'm taking over on @mpickering's request. |
@mergify rebase |
✅ Branch has been successfully rebased |
7e3a32b
to
61de5ee
Compare
@mergify rebase |
Co-authored-by: Artem Pelenitsyn <[email protected]>
✅ Branch has been successfully rebased |
61de5ee
to
822997d
Compare
A windows validate job failed for vaguely alarming reasons, that maybe fixed on master by now. I haven't investigated further, but restarted. If this recurs on master, we'd need to investigate. |
@mergify backport master |
✅ Backports have been created
|
* Add release notes for 3.14.2.0 release (cherry picked from commit 7bd6198) # Conflicts: # Cabal-hooks/CHANGELOG.md # cabal-install/changelog # changelog.d/pr-10493 # changelog.d/pr-10546 # changelog.d/pr-10828.md * Remove redundant Other Changes header Fixes #10685 (cherry picked from commit b2606fe) * Apply suggestions from code review Co-authored-by: Artem Pelenitsyn <[email protected]> (cherry picked from commit 822997d) * fix conflicts * typos * remove little changelog files that somehow Mergify failed to remove during cherry-pick * add the missing new entry in cabal-install/ChangeLog.md --------- Co-authored-by: Matthew Pickering <[email protected]> Co-authored-by: Mikolaj Konarski <[email protected]> Co-authored-by: Artem Pelenitsyn <[email protected]>
Add release notes for the 3.14.2.0 release