Skip to content

Conversation

@securebitsorg
Copy link
Owner

No description provided.

marci added 2 commits September 8, 2025 06:51
- Document that branch protection errors are expected and correct
- Explain why direct pushes to main are blocked (security feature)
- Provide multiple solution options for release workflow
- Recommend workflow adjustments over disabling protection
- Clarify that this is a success, not a failure
…nch protection

- Remove git push origin main from auto-release-on-push workflow
- Keep only tag push (git push origin v$NEW_VERSION) which is allowed
- Add documentation explaining the fix and why it's better
- Ensure workflow respects branch protection rules while maintaining functionality
- Tags can be pushed independently without touching main branch
- GitHub Releases are created from tags, not main branch commits
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, please upgrade to Bugbot Pro by visiting the Cursor dashboard. Your first 14 days will be free!

# Tag erstellen und pushen (ohne main-Branch zu berühren)
git tag -a "v$NEW_VERSION" -m "Automatic release v$NEW_VERSION"
git push origin main
echo "Pushing tag v$NEW_VERSION (no main branch push needed)"
Copy link

Choose a reason for hiding this comment

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

Bug: Version File Updates Not Pushed to Main

The workflow commits version file updates (VERSION, __version__.py, pyproject.toml) locally but no longer pushes these changes to the main branch. This leaves main's version files outdated, which can lead to incorrect future version calculations. It also creates an inconsistent state where the release tag references a commit not present on the remote main branch.

Additional Locations (1)

Fix in Cursor Fix in Web

@securebitsorg securebitsorg merged commit 0824471 into main Sep 8, 2025
14 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