Skip to content

fix(ci): use correct label name in screenshots workflow#1430

Open
jeremyeder wants to merge 3 commits intomainfrom
fix/screenshots-label-typo
Open

fix(ci): use correct label name in screenshots workflow#1430
jeremyeder wants to merge 3 commits intomainfrom
fix/screenshots-label-typo

Conversation

@jeremyeder
Copy link
Copy Markdown
Contributor

@jeremyeder jeremyeder commented Apr 22, 2026

Summary

Test plan

  • Next scheduled run of the screenshots workflow creates the PR without label errors

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated automated workflow configuration

The workflow used --label "docs" but the repo label is "documentation".
This caused the scheduled screenshot PR creation to fail with
"could not add label: 'docs' not found".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 06254a7
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/69e8eaba9191340007dc3886

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 7963c15
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/69e977035d90e1000887c003

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 22, 2026

📝 Walkthrough

Walkthrough

Updates a GitHub Actions workflow to change PR labels from docs,automated to documentation,automated in the automated screenshot update step. This is a label string modification in the workflow configuration.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/screenshots.yml
Updated PR labels in the "Create PR with updated screenshots" step from docs,automated to documentation,automated.
🚥 Pre-merge checks | ✅ 8
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title uses Conventional Commits format (fix(ci): ...) and accurately describes the change: correcting an incorrect label name in the screenshots workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Performance And Algorithmic Complexity ✅ Passed Pull request modifies only GitHub Actions workflow, changing label from 'docs' to 'documentation'. No performance-sensitive changes detected.
Security And Secret Handling ✅ Passed PR modifies label from 'docs' to 'documentation,automated' in GitHub Actions workflow. No secrets hardcoded, exposed, or logged; proper use of GitHub's built-in secrets.GITHUB_TOKEN and test mocks; no injection vulnerabilities.
Kubernetes Resource Safety ✅ Passed Kubernetes Resource Safety check is not applicable; PR only modifies GitHub Actions workflow configuration file, not Kubernetes manifests.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/screenshots-label-typo
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch fix/screenshots-label-typo

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.github/workflows/screenshots.yml (1)

24-24: 🛠️ Refactor suggestion | 🟠 Major

Pin all actions to specific commit SHAs.

The workflow uses semantic version tags (@v6, @master) instead of commit SHAs for actions/checkout, actions/setup-node, actions/upload-artifact, and kubeflow/pipelines. This introduces supply chain risk if tags are moved or compromised. Pin to these SHAs:

  • actions/checkout@v6de0fac2e4500dabe0009e67214ff5f5447ce83dd
  • actions/setup-node@v648b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
  • actions/upload-artifact@v6b7c566a772e6b6bfb58ed0dc250532a479d7789f
  • kubeflow/pipelines/.github/actions/github-disk-cleanup@master8b27aca612c83881de75baf6815fdd566b154ef3

Applies to lines 24, 27, 30, and 113.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/screenshots.yml at line 24, Replace the loose semantic
tags with the provided commit SHAs to pin the actions and remove supply-chain
risk: update actions/checkout@v6 to
actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd, actions/setup-node@v6
to actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e,
actions/upload-artifact@v6 to
actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f, and
kubeflow/pipelines/.github/actions/github-disk-cleanup@master to
kubeflow/pipelines/.github/actions/github-disk-cleanup@8b27aca612c83881de75baf6815fdd566b154ef3
so the workflow references immutable SHAs instead of moving tags.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In @.github/workflows/screenshots.yml:
- Line 24: Replace the loose semantic tags with the provided commit SHAs to pin
the actions and remove supply-chain risk: update actions/checkout@v6 to
actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd, actions/setup-node@v6
to actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e,
actions/upload-artifact@v6 to
actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f, and
kubeflow/pipelines/.github/actions/github-disk-cleanup@master to
kubeflow/pipelines/.github/actions/github-disk-cleanup@8b27aca612c83881de75baf6815fdd566b154ef3
so the workflow references immutable SHAs instead of moving tags.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 52a6b725-e44f-4e96-8930-5a52a4e5447c

📥 Commits

Reviewing files that changed from the base of the PR and between 341747a and 06254a7.

📒 Files selected for processing (1)
  • .github/workflows/screenshots.yml

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