Skip to content

fix: add shell: bash to version extraction steps#23

Merged
thoroc merged 1 commit intomainfrom
fix/release-please-outputs
Feb 7, 2026
Merged

fix: add shell: bash to version extraction steps#23
thoroc merged 1 commit intomainfrom
fix/release-please-outputs

Conversation

@thoroc
Copy link
Owner

@thoroc thoroc commented Feb 7, 2026

Problem\nThe "Extract version from Cargo.toml" step failed on Windows with:\n\n\nThe module 'VERSION=$(grep -E '^version' could not be loaded.\n\n\nThis is because Windows runners default to PowerShell, not bash.\n\n## Solution\nAdded shell: bash to both "Extract version from Cargo.toml" steps to ensure the bash commands run correctly across all platforms (Linux, macOS, and Windows).

@thoroc thoroc force-pushed the fix/release-please-outputs branch from c025965 to f32e621 Compare February 7, 2026 05:14
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

@github-copilot Please review this PR focusing on:

  1. Code quality, best practices, and potential bugs
  2. Security vulnerabilities and unsafe patterns
  3. Performance implications
  4. Documentation accuracy and completeness
  5. Test coverage and edge cases

Please provide specific, actionable feedback.

@thoroc thoroc merged commit 01a749d into main Feb 7, 2026
1 check passed
@thoroc thoroc deleted the fix/release-please-outputs branch February 7, 2026 05:15
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