Skip to content

ci: bump pr_style_bot to current style-bot-action reusable workflow - #1360

Open
Abelo9996 wants to merge 1 commit into
huggingface:mainfrom
Abelo9996:fix-pr-style-bot-reusable-wf
Open

ci: bump pr_style_bot to current style-bot-action reusable workflow#1360
Abelo9996 wants to merge 1 commit into
huggingface:mainfrom
Abelo9996:fix-pr-style-bot-reusable-wf

Conversation

@Abelo9996

Copy link
Copy Markdown

What

Bump the pinned reusable workflow in .github/workflows/pr_style_bot.yaml from huggingface_hub/.github/workflows/style-bot-action.yml@42642d4a to @9a3222e.

Why

The current pin fails to start on issue_comment events, so every comment on an issue or PR produces a failed "PR Style Bot" run (for example: https://github.com/huggingface/lighteval/actions/runs/33112348806). The run creates no jobs, which indicates a startup failure resolving the reusable workflow at the old pin. That version bundles outdated actions (github-script v8, checkout v6, upload-artifact v4.6.0, download-artifact v4.1.8).

9a3222e is the current version of that reusable workflow on huggingface_hub main, with those actions bumped (github-script v9, checkout v7, upload-artifact v7, download-artifact v8). The workflow_call interface is identical (python_quality_dependencies input, app_id and app_private_key secrets), so the caller needs no other changes.

Note

issue_comment workflows only run from the default branch, so this cannot be exercised from a PR branch and will take effect once merged to main.

The PR Style Bot workflow pins huggingface_hub's style-bot-action.yml at
42642d4a, whose bundled actions are outdated (github-script v8, checkout v6,
upload-artifact v4.6.0, download-artifact v4.1.8). The reusable workflow fails
to start on issue_comment events, so every issue comment produces a failed run.

Bump the pin to 9a3222e, the current version on huggingface_hub main
(github-script v9, checkout v7, upload-artifact v7, download-artifact v8). The
workflow_call interface is unchanged, so no caller changes are needed.
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