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

8.0.0 changelog #2600

Merged
merged 1 commit into from
Apr 14, 2024
Merged

8.0.0 changelog #2600

merged 1 commit into from
Apr 14, 2024

Conversation

dgw
Copy link
Member

@dgw dgw commented Mar 16, 2024

Since I only wanted to work on this if I could do a whole page of 25 PRs in one sitting, it sometimes took over a week between sessions. So here we are in March despite me starting on this in earnest two months ago.

I don't know how far along @SnoopJ has gotten with putting feedback on sopel-irc/sopel.chat#46 but this and that should probably be reviewed somewhat together. The changelog (this) summarizes the highlights, and the upgrade guide (that) gives all the details with suggested replacements for deprecated/removed features.

This will be a lesson for future changelogs, and we're going to have an 8.1-NEWS, 9.0-NEWS, etc. branch for the next releases. I've learned that the way we did it for 7.0.0 was Good, Actually!

And yes, I am very very tempted to add this PR itself to the changelog. We like being meta around here, right?

@dgw dgw added this to the 8.0.0 milestone Mar 16, 2024
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

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

I think it's ready to squash and merge. I mean... I didn't see any typo or missed linked, but at this point it's kinda hard to tell with so many things!

Well done. 🎉

@dgw dgw marked this pull request as ready for review April 14, 2024 15:56
Copy link
Contributor

@Exirel Exirel left a comment

Choose a reason for hiding this comment

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

Nothing to add. It's already so much!

@dgw
Copy link
Member Author

dgw commented Apr 14, 2024

Since we decided (on IRC a few weeks ago) that any omissions in the upgrade guide can be fixed later, it's not that much different to treat omissions or errors in the changelog the same way. Anyone referring to the changelog later is welcome to point out broken or mixed-up links, etc. in lieu of getting a second line-by-line review here and now.

The website can be updated without needing to upload a new version, but we also have the option for post releases if any significant mistake can't wait until the next patch release to be included in the distribution.

@dgw dgw merged commit ab32aca into master Apr 14, 2024
13 checks passed
@dgw dgw deleted the 8.0-NEWS branch April 14, 2024 16:15
@dgw dgw mentioned this pull request Apr 24, 2024
4 tasks
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