Skip to content

fix: add missing permissions to release workflow#221

Merged
darvid merged 6 commits intomainfrom
fix/release-workflow-permissions
Aug 8, 2025
Merged

fix: add missing permissions to release workflow#221
darvid merged 6 commits intomainfrom
fix/release-workflow-permissions

Conversation

@darvid
Copy link
Copy Markdown
Owner

@darvid darvid commented Aug 7, 2025

fixes reusable workflow permissions error in release.yml that was blocking builds after adding explicit workflow permissions

darvid added 5 commits August 7, 2025 16:35
Fix reusable workflow permissions error in release.yml that was
blocking builds after adding explicit workflow permissions.
- Add actionlint to pre-commit hooks for local validation
- Add actionlint step to lint workflow for CI validation
- Prevents workflow errors from reaching main branch
Remove double curly braces around vars.RELEASE_PR_BRANCH that was
causing condition to always evaluate to true.
- Quote shell variables properly to prevent globbing/word splitting
- Address SC2086 warnings in build, publish, and release workflows
Use raven-actions/actionlint@v2 from marketplace
@darvid darvid force-pushed the fix/release-workflow-permissions branch 2 times, most recently from b13514b to fce1d94 Compare August 7, 2025 21:03
@darvid darvid force-pushed the fix/release-workflow-permissions branch from fce1d94 to c47ad45 Compare August 7, 2025 21:11
@darvid darvid merged commit 7a4b410 into main Aug 8, 2025
42 checks passed
@darvid darvid deleted the fix/release-workflow-permissions branch October 21, 2025 15:51
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