Skip to content

Fix deprecation in CI Sync TRL skills#6200

Merged
albertvillanova merged 1 commit into
mainfrom
fix-deprecation-skills-sync
Jun 30, 2026
Merged

Fix deprecation in CI Sync TRL skills#6200
albertvillanova merged 1 commit into
mainfrom
fix-deprecation-skills-sync

Conversation

@albertvillanova

@albertvillanova albertvillanova commented Jun 29, 2026

Copy link
Copy Markdown
Member

Fix deprecation in CI Sync TRL skills.

Currently, a warning is emitted by GH Action "Sync TRL skill with huggingface/skills", step "Create GitHub App token" using "actions/create-github-app-token" GH Action: https://github.com/huggingface/trl/actions/runs/28205349188/job/83554678491

Create GitHub App token
Input 'app-id' has been deprecated with message: Use 'client-id' instead.

This PR updates the input parameter for the actions/create-github-app-token action, replacing deprecated app-id with client-id in the .github/workflows/sync-huggingface-skills.yml file.


Note

Low Risk
One-line CI workflow input rename with the same secret value; no application or security logic changes.

Overview
Updates the Sync TRL skill with huggingface/skills workflow so the actions/create-github-app-token step uses client-id instead of the deprecated app-id input.

The secret name (APP_ID_HUB_SKILLS_REPO) is unchanged; only the action input key matches the current API and removes the CI warning.

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

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

@albertvillanova albertvillanova merged commit e890235 into main Jun 30, 2026
21 of 24 checks passed
@albertvillanova albertvillanova deleted the fix-deprecation-skills-sync branch June 30, 2026 06:08
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