Skip to content

fix: allow empty commit when preparing release PR#733

Merged
thomasrockhu-codecov merged 2 commits into
mainfrom
th/allow-empty-release-commit
Jul 8, 2026
Merged

fix: allow empty commit when preparing release PR#733
thomasrockhu-codecov merged 2 commits into
mainfrom
th/allow-empty-release-commit

Conversation

@thomasrockhu-codecov

Copy link
Copy Markdown
Contributor

Summary

  • The Create CLI Release PR workflow failed with nothing to commit, working tree clean (e.g. run 28972926678) when the version in pyproject.toml already matched the requested release version, so sed produced no diff and git commit exited non-zero.
  • Add --allow-empty to the release commit so the release branch and PR are still created even when there is no version change.

Test plan

  • Run the Create CLI Release PR workflow with a version that already matches pyproject.toml and confirm the branch/PR is created with an (empty) signed commit.

Made with Cursor

When the version in pyproject.toml already matches the requested release
version, sed makes no change and `git commit` fails with "nothing to commit".
Add --allow-empty so the release branch/PR is still created.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
723 1 722 1
View the full list of 1 ❄️ flaky test(s)
api.temp.calculator.test_calculator::test_divide

Flake rate in main: 100.00% (Passed 0 times, Failed 1057 times)

Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@thomasrockhu-codecov thomasrockhu-codecov merged commit c901634 into main Jul 8, 2026
22 checks passed
@thomasrockhu-codecov thomasrockhu-codecov deleted the th/allow-empty-release-commit branch July 8, 2026 20:33
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