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

Prepare for release of version 5.3.0 of apptools #353

Merged
merged 3 commits into from
Jul 5, 2024
Merged

Conversation

mdickinson
Copy link
Member

This PR (against the main branch) bumps the version and updates the changelog in preparation for a 5.3.0 release of Apptools. The plan is to tag the merge commit from the eventual squash-merge of this PR.

@flongford Would you have bandwidth to review?

Copy link

@flongford flongford left a comment

Choose a reason for hiding this comment

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

LGTM, though with the disclaimer that I'm not very familiar with the current release process.

Nice to see how the scriv snippets contribute to the release notes!

@mdickinson
Copy link
Member Author

@flongford Thanks for review!

I've just discovered that there are release notes for 5.2.1 on the 5.2.1 tag, but that they weren't ever merged back into the main branch (and they have an oddly incorrect release date). I've done my best to reconcile those with the ones introduced in this PR.

I'm not very familiar with the current release process

There isn't really one, or at least not one that I'd care to imitate. We had a somewhat over-elaborate process for previous releases, and I'm doing something simpler this time around (in particular, since this is a minor release rather than a patch release, I'm releasing straight from the main branch). I'll add some notes on the process to the (currently empty) wiki, for possible re-use next time.

Nice to see how the scriv snippets contribute to the release notes!

Yep, this was our own home-grown approximation to scriv (from a time that predated scrivs existence), introduced by @kitchoi. Now that scriv exists, I think we should probably switch to using that from this point onwards.

@mdickinson
Copy link
Member Author

I'll add some notes on the process to the (currently empty) wiki

Started some notes here: https://github.com/enthought/apptools/wiki/Releasing

@mdickinson mdickinson merged commit 8e4935d into main Jul 5, 2024
24 checks passed
@mdickinson mdickinson deleted the release/5.3.0 branch July 5, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants