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

Update version pull req #1022

Merged
merged 5 commits into from
Feb 28, 2025
Merged

Update version pull req #1022

merged 5 commits into from
Feb 28, 2025

Conversation

umbertoDifa
Copy link
Collaborator

Fix version init position

@Copilot Copilot bot review requested due to automatic review settings February 28, 2025 14:00
@umbertoDifa umbertoDifa merged commit a5f787e into master Feb 28, 2025
7 checks passed
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This pull request updates the version bump workflow by repositioning the init file and updating the configuration for creating pull requests.

  • Changed the file path in the git add command to use darwin/version/init.py
  • Upgraded the peter-evans/create-pull-request action from v5 to v7 and updated its parameter name

Reviewed Changes

File Description
.github/workflows/version_bump.yml Adjusted the file path for the version init and updated the pull request action config

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:115

  • Ensure that 'darwin/version/init.py' is the intended new file path and that related import paths in the project are updated accordingly.
git add pyproject.toml darwin/version/__init__.py

.github/workflows/version_bump.yml:127

  • Verify that the updated 'branch' parameter aligns with v7 requirements of the create-pull-request action, replacing the previous 'head' parameter.
branch: ${{ env.BRANCH_NAME }}

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