Skip to content

fix(release.yml): restore env: block on 'Detect version' step#4267

Merged
Wauplin merged 1 commit into
mainfrom
fix/release-yml-restore-env-block
May 26, 2026
Merged

fix(release.yml): restore env: block on 'Detect version' step#4267
Wauplin merged 1 commit into
mainfrom
fix/release-yml-restore-env-block

Conversation

@paulinebm

@paulinebm paulinebm commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary

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>
@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 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.

sorry merging too fast 🙈

@Wauplin Wauplin merged commit 34cc3fa into main May 26, 2026
15 of 22 checks passed
@Wauplin Wauplin deleted the fix/release-yml-restore-env-block branch May 26, 2026 14:23
@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.

2 participants