Skip to content

Fix sentry-cli path on Windows Git Bash#176

Merged
fernandotonon merged 1 commit intomasterfrom
fix/windows-sentry-cli-path
Mar 6, 2026
Merged

Fix sentry-cli path on Windows Git Bash#176
fernandotonon merged 1 commit intomasterfrom
fix/windows-sentry-cli-path

Conversation

@fernandotonon
Copy link
Owner

@fernandotonon fernandotonon commented Mar 6, 2026

Summary

  • The curl sentry-cli installer places the binary at /usr/local/bin/sentry-cli, which is not in Git Bash's PATH on Windows runners
  • Use the full path /usr/local/bin/sentry-cli instead of just sentry-cli

Test plan

  • Merge and verify Windows release build passes

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated deployment workflow configuration for improved reliability.

The curl installer places sentry-cli in /usr/local/bin which is not in
Git Bash's PATH on Windows runners. Use the full path instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6543792d-7883-4c33-9453-65399ea11ced

📥 Commits

Reviewing files that changed from the base of the PR and between 6588ce5 and 710e2e5.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yml

📝 Walkthrough

Walkthrough

The GitHub Actions deploy workflow is updated to invoke the sentry-cli binary using its absolute path (/usr/local/bin/sentry-cli) instead of relying on command-line PATH resolution when uploading debug files to Sentry.

Changes

Cohort / File(s) Summary
Sentry CLI invocation
.github/workflows/deploy.yml
Replaces relative sentry-cli command with absolute path /usr/local/bin/sentry-cli for debug file uploads.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A path once lost in shadowed lands,
Now gleams in absolute commands,
No search through winding PATH we go,
Just /usr/local/bin—clear and true! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: using an absolute path for sentry-cli on Windows Git Bash instead of relying on PATH resolution.
Description check ✅ Passed The description is missing the 'Technical Details' section with bullets of areas changed, and the checkboxes are incomplete. However, the Summary section is clear and the overall intent is understandable.
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
  • Commit unit tests in branch fix/windows-sentry-cli-path

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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2026

@fernandotonon fernandotonon merged commit 34c7f51 into master Mar 6, 2026
15 checks passed
@fernandotonon fernandotonon deleted the fix/windows-sentry-cli-path branch March 6, 2026 01:46
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.

1 participant