Skip to content

Conversation

TTOzzi
Copy link
Member

@TTOzzi TTOzzi commented Sep 19, 2025

This retries the work attempted in #975 and #1029.
When a release branch is created, this workflow automatically updates the version output value in PrintVersion.swift to match the release version, commits the change, and opens a PR.

If committing directly to the release branch is acceptable, that would be cleaner. Please let me know your thoughts, and I’ll adjust the workflow accordingly.

Copy link
Contributor

@bnbarham bnbarham left a comment

Choose a reason for hiding this comment

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

Ooo, interesting idea! We do still bump without branch creation, but this would at least catch some of them.

Thoughts @shahmishal?

Comment on lines 54 to 56
--body "This PR updates the version in PrintVersion.swift to ${{ env.VERSION }}.

Automatically created when release branch ${{ env.RELEASE_BRANCH }} was created." \
Copy link
Contributor

Choose a reason for hiding this comment

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

How about the following:

This PR was automatically opened by a GitHub action on creation of a release branch (${{ env.RELEASE_BRANCH }}). Review the version update and merge if correct, otherwise update the version manually.

?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I’ve updated it. Thank you!

@TTOzzi TTOzzi force-pushed the update-version-action branch from dd655d6 to 5d7e295 Compare September 20, 2025 08:11
@TTOzzi TTOzzi force-pushed the update-version-action branch from 5d7e295 to 3e6a24d Compare September 20, 2025 08:12
@TTOzzi
Copy link
Member Author

TTOzzi commented Sep 20, 2025

Ah, so in cases like 6.1.2 and 6.1.3, they were released with just tags and no branch 🫨

@bnbarham
Copy link
Contributor

bnbarham commented Sep 22, 2025

they were released with just tags and no branch 🫨

Indeed 😅 I still think this is better than what we currently have - at least we can't miss major version bumps.

The Windows failures are fixed by swiftlang/swift-markdown#245, but it hasn't been merged yet.

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.

3 participants