Skip to content

Conversation

@Git-HimanshuRathi
Copy link
Contributor

@Git-HimanshuRathi Git-HimanshuRathi commented Jan 31, 2026

Summary

The manual desktop release workflow was failing because it attempted to push directly to main, which is protected.
This PR updates the workflow to use a release branch + PR-based flow, fully respecting branch protection rules.

Fixes #903


What changed

  • Create a versioned release branch (release/vX.Y.Z) instead of pushing to the target branch
  • Automatically open a PR from the release branch to the triggering branch
  • Skip PR creation if one already exists (safe re-runs)
  • Run build and release jobs against the release branch
  • Add pull-requests: write permission
  • Document tag creation behavior in the auto-generated PR

Why this approach

  • Works with protected branches
  • Keeps release changes auditable via PRs
  • Matches GitHub’s recommended workflow
  • Avoids force pushes or bypassing rules

Code Understanding and AI Usage

  • Yes, I used AI assistance
  • I reviewed and understand every line of the code
  • I tested the workflow and considered edge cases

Checklist

  • Linked the issue
  • Self-reviewed the code
  • Changes follow project conventions
  • I can explain all added logic

Summary by CodeRabbit

  • Chores
    • Improved release workflow with enhanced version management and automated pull request handling for releases.

✏️ Tip: You can customize this high-level summary in your review settings.

…tection

- Add pull-requests: write permission
- Create release branch instead of pushing directly to target branch
- Automatically create PR from release branch to target branch
- Check if PR already exists before creating (idempotent re-runs)
- Update build-tauri and create-release jobs to use release branch
- Document tag timing behavior in auto-generated PR description

This fixes the issue where releases fail on protected branches that
require PRs for all changes.
@netlify
Copy link

netlify bot commented Jan 31, 2026

Deploy Preview for circuitverse ready!

Name Link
🔨 Latest commit d9ac7bf
🔍 Latest deploy log https://app.netlify.com/projects/circuitverse/deploys/697f51f83322bb000876afa7
😎 Deploy Preview https://deploy-preview-904--circuitverse.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 47 (🔴 down 1 from production)
Accessibility: 73 (no change from production)
Best Practices: 92 (no change from production)
SEO: 82 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 31, 2026

Walkthrough

The pull request modifies the Tauri release workflow in .github/workflows/Tauri-Release.yml to implement a release branch strategy. It adds pull-request write permissions, introduces a release_branch output from the version job, and restructures the workflow to create a release branch (release/$NEW_VERSION) and open a pull request targeting the original branch instead of committing directly to the main branch. Subsequent workflow steps are updated to reference this generated release branch through needs.version.outputs.release_branch rather than using github.ref_name, ensuring the entire pipeline operates on the release branch.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title references the main change (release branch and PR for version bump) and directly relates to the changeset's primary objective of respecting branch protection rules.
Linked Issues check ✅ Passed All coding requirements from issue #903 are met: release branch creation, PR opening from release branch, and workflow updates to use the release branch instead of direct pushes.
Out of Scope Changes check ✅ Passed All changes directly address the linked issue requirements; no unrelated modifications to functionality or structure were introduced.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ThatDeparted2061
Copy link
Member

attach some screenshots please

Conventional Changelog Action and others added 2 commits February 1, 2026 12:39
@Git-HimanshuRathi
Copy link
Contributor Author

@Git-HimanshuRathi
Copy link
Contributor Author

version got bumped , reverted it back to 0.0.0 .

@Git-HimanshuRathi
Copy link
Contributor Author

@ThatDeparted2061 @tachyons ready for review .

@ThatDeparted2061
Copy link
Member

Nice good work

@ThatDeparted2061
Copy link
Member

@tachyons LGTM

@Git-HimanshuRathi
Copy link
Contributor Author

@tachyons PR has been approved by harsh .

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 desktop release workflow failing due to protected main branch

2 participants