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

Releases: Replace versions in upgrade scripts #1179

Merged
merged 2 commits into from
Jan 20, 2025
Merged

Conversation

obenland
Copy link
Member

See #1175 (comment)

If someone were to release manually I guess the worst that would happen would be that the upgrade routine never runs.

Proposed changes:

  • Adds class-migration.php to the list of files the release script looks at to replace version numbers.
  • Adds instructional comment in class-migration.php to surface that behavior.

Closes #1162.

@obenland obenland added the Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary. label Jan 17, 2025
@obenland obenland requested a review from a team January 17, 2025 17:14
@obenland obenland self-assigned this Jan 17, 2025
@obenland obenland merged commit cc99cde into trunk Jan 20, 2025
20 checks passed
@obenland obenland deleted the update/release-script branch January 20, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants