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

Version bump update #1018

Merged
merged 2 commits into from
Feb 28, 2025
Merged

Version bump update #1018

merged 2 commits into from
Feb 28, 2025

Conversation

umbertoDifa
Copy link
Collaborator

Automatically create a PR with the version bump and a draft release

@Copilot Copilot bot review requested due to automatic review settings February 28, 2025 10:44

Choose a reason for hiding this comment

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

PR Overview

This PR automates the version bump release process by updating the GitHub Actions workflow. Key changes include:

  • The removal of the pull_request trigger.
  • Dynamic branch naming for version releases.
  • Addition of a step to automatically create a pull request to the master branch.

Reviewed Changes

File Description
.github/workflows/version_bump.yml Updated workflow to automate branch creation, commit, PR, tagging, and release for version bumps

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

.github/workflows/version_bump.yml:98

  • [nitpick] The branch naming format 'release-${TAG_PREFIX}${NEW_VERSION}' may lead to collisions if multiple version bumps occur concurrently. Consider incorporating a unique identifier or timestamp to ensure branch name uniqueness.
BRANCH_NAME="release-${TAG_PREFIX}${NEW_VERSION}"

.github/workflows/version_bump.yml:117

  • This step unconditionally pushes the branch regardless of the TEST_MODE environment. Please verify that this behavior is intended for test mode workflows, or consider adding an appropriate condition.
git push origin ${BRANCH_NAME}
@umbertoDifa umbertoDifa merged commit 63b2aed into master Feb 28, 2025
7 checks passed
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