Skip to content

Release Action: Pin GitPython - #472

Closed
annehaley wants to merge 1 commit into
masterfrom
release-action-pin
Closed

Release Action: Pin GitPython#472
annehaley wants to merge 1 commit into
masterfrom
release-action-pin

Conversation

@annehaley

Copy link
Copy Markdown
Collaborator

Addresses this CI failure.

GitPython just released 3.1.60, which includes a breaking change with the removal of Actor.name_email_regex, which is needed by python semantic release. (This commit has the change).

This PR pins GitPython to 3.1.59 in the release action.

@annehaley
annehaley requested a review from BryonLewis August 25, 2026 19:53

@BryonLewis BryonLewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this is fine for now. I would watch the issue on python-semantic-release: python-semantic-release/python-semantic-release#1476
Looks like there is a PR an fix already so an upgrade to it in the future should prevent needing to pin GitPython. Although I'm not sure if the new fix of just directly recreating that regular expression avoids the fundamental reason why it was removed in GitPython.

@annehaley

Copy link
Copy Markdown
Collaborator Author

Yeah with the creation of that PR in python-semantic-release, I'll wait a bit before merging this. If the upstream is fixed quickly, then we can just rerun the job and close this if it succeeds.

@annehaley

Copy link
Copy Markdown
Collaborator Author

GitPython released 3.1.61 this morning, which brings back (but deprecates) Actor.name_email_regex. Our release job works again. The discussion on the python-semantic-release PR linked above will decide how to deal with the future removal.

@annehaley annehaley closed this Aug 28, 2026
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