[actions] fix labels removing

[capcom6]

Summary by CodeRabbit

• Chores
  • Improved CI/CD workflow stability with enhanced error handling for label management processes, ensuring robust execution of sequential operations without interruption due to individual command failures.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[cloudflare-workers-and-pages]

Deploying docs with    Cloudflare Pages

Latest commit:	1991600
Status:	✅  Deploy successful!
Preview URL:	https://115567d0.docs-7wm.pages.dev
Branch Preview URL:	https://actions-fix-labels-removing.docs-7wm.pages.dev

View logs

[coderabbitai]

Walkthrough

The PR modifies a GitHub Actions workflow file to refactor label-removal logic. It introduces a separate checkout step and splits a single gh pr edit command into two sequential commands, each removing one label independently with non-fatal error handling.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow Refactoring \.github/workflows/pr-labels\.yml	Added explicit checkout step at job start; split single label-removal command into two sequential gh pr edit calls (removing "ready" and "deployed" labels separately) with non-fatal error handling (

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Verify that the separate checkout step correctly initializes the repository context
• Confirm that splitting the label-removal commands into two operations maintains the intended behavior
• Check that the non-fatal error handling (|| true) allows both removals to attempt completion without blocking subsequent steps

Possibly related PRs

• [actions] add PR labels clenup on update #153: Modifies the same workflow file to remove "ready" and "deployed" labels; this PR refactors that approach by introducing a checkout step and splitting the label-removal into two sequential commands with error handling.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title '[actions] fix labels removing' directly relates to the main change: fixing the label removal process in the GitHub Actions workflow by splitting commands and adding proper error handling.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch actions/fix-labels-removing

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (2)

.github/workflows/pr-labels.yml (2)

13-15: Checkout step may be unnecessary for this workflow.

The actions/checkout@v4 step is not required in this context. Since the workflow only uses the gh CLI to remove labels via the GitHub API (and doesn't depend on any repository code), this step adds unnecessary overhead. The gh CLI operates directly through the GitHub API and does not need a local checkout to function.

Consider removing this step to reduce workflow execution time.

---

19-21: Consider improving error handling and observability.

The || true approach silently suppresses all errors, including potential authentication issues or unexpected failures. While this pragmatically handles cases where a label doesn't exist on a PR, it reduces observability and makes troubleshooting harder.

Consider a more explicit approach:

• Add conditional checks or logging to understand why a command failed
• Use a helper script that distinguishes between "label doesn't exist" (expected, safe to ignore) and other failures (worth logging)
• Or at minimum, log the output to aid in debugging

This is less critical if label-removal failures are truly expected and non-critical, but better observability would improve maintainability.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dbc2def and 1991600.

📒 Files selected for processing (1)

• .github/workflows/pr-labels.yml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Cloudflare Pages