Skip to content

[WIP] Fix release CI#12

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-9
Closed

[WIP] Fix release CI#12
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-9

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 21, 2025

  • Analyze repository structure and understand the issues
  • Identify the regex pattern issue in tool/bump_version.py
  • Identify the git rebase issue in .github/workflows/release.yml
  • Fix the regex pattern in _replace_version function to use proper group reference
  • Fix the CI workflow to handle git state properly before rebase
  • Test the fixes to ensure they work correctly
  • Validate the changes don't break existing functionality

Issues Found:

  1. Regex Issue: The rf"\g<1>{new_version}" pattern in _replace_version function causes "invalid group reference 10" when new_version contains digits that get concatenated with the group reference (e.g., \g<1>1.0.10 becomes \g<110>).

  2. Git Rebase Issue: The release CI workflow tries to rebase after making commits, but the script modifies files without properly managing git state, causing "You have unstaged changes" error.

Plan:

  • Fix the regex pattern by separating the group reference from the variable substitution
  • Update the CI workflow to properly handle git state management around the rebase operation

Fixes #9.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

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.

Fix release CI

2 participants