Skip to content

chore: update release.yml#4264

Merged
Wauplin merged 1 commit into
mainfrom
security-fix/-github-workflows-release-yml-1779802593
May 26, 2026
Merged

chore: update release.yml#4264
Wauplin merged 1 commit into
mainfrom
security-fix/-github-workflows-release-yml-1779802593

Conversation

@hf-security-analysis

@hf-security-analysis hf-security-analysis Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

Update .github/workflows/release.yml workflow configuration.

cc @dependabot[bot] @Wauplin

Closes huggingface/tracking-issues#779


Note

Low Risk
Single-line CI wiring change with no effect on version logic or publish paths.

Overview
The Detect version step in .github/workflows/release.yml no longer sets RELEASE_TYPE via a step-level env block. It now assigns RELEASE_TYPE inside the run script with RELEASE_TYPE="${{ inputs.release_type }}", matching other steps in the same workflow that inject dispatch inputs directly into shell.

Reviewed by Cursor Bugbot for commit 42dfa94. Bugbot is set up for automated code reviews on this repo. Configure here.

@hf-security-analysis hf-security-analysis Bot requested a review from paulinebm May 26, 2026 13:36

@Wauplin Wauplin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Trusting the bot it's best like this ^^

@Wauplin Wauplin merged commit 1685207 into main May 26, 2026
18 of 22 checks passed
@Wauplin Wauplin deleted the security-fix/-github-workflows-release-yml-1779802593 branch May 26, 2026 13:38
@bot-ci-comment

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Wauplin pushed a commit that referenced this pull request May 26, 2026
PR #4264 removed the env: block on the 'Detect version' step and inlined
`RELEASE_TYPE="${{ inputs.release_type }}"` directly into the run script.
That patch was produced by an autofix bot in response to a false-positive
command_injection finding; the env-block pattern that was already in place
is the project's recommended mitigation, so the autofix was a regression.

Restoring the env: block restores the consistent convention for this repo
(all ${{ ... }} expressions used inside a run: script are first mapped to
shell variables via env:).

Tracking: huggingface/tracking-issues#780

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@huggingface-hub-bot

Copy link
Copy Markdown
Contributor

This PR has been shipped as part of the v1.17.0 release.

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