Skip to content

[Internal] Use Comment bot in model card gh action#4277

Merged
Wauplin merged 3 commits into
mainfrom
use-comment-bot-in-model-card-gh-action
May 27, 2026
Merged

[Internal] Use Comment bot in model card gh action#4277
Wauplin merged 3 commits into
mainfrom
use-comment-bot-in-model-card-gh-action

Conversation

@Wauplin

@Wauplin Wauplin commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Should solve https://github.com/huggingface/tracking-issues/issues/781. To be merged instead of #4276.

Once merged we can remove comment_bot_token from this repo settings.

cc @paulinebm


Note

Low Risk
Only GitHub Actions credential wiring changes; no runtime library or user-facing behavior is modified.

Overview
The model and dataset card consistency reminder workflow now authenticates PR comments with a GitHub App instead of a long-lived comment_bot_token secret.

A new Create GitHub App token step uses actions/create-github-app-token with COMMENT_BOT_APP_ID and COMMENT_BOT_SECRET_PEM, scoped to the repo owner and current repository. The sticky comment step’s GITHUB_TOKEN is wired to that minted token. The reminder message and path triggers are unchanged.

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

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

Ok for me 👍

Comment thread .github/workflows/model_card_consistency_reminder.yml Outdated
@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 merged commit 76a9742 into main May 27, 2026
22 of 26 checks passed
@Wauplin Wauplin deleted the use-comment-bot-in-model-card-gh-action branch May 27, 2026 09:19
@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.

3 participants