Skip to content

ci: pin PR labeler to allowlisted 1.3.0 commit#265

Merged
svvarom merged 1 commit into
mainfrom
aireinf-208-labeler-allowlist-pin/smuley
Jun 10, 2026
Merged

ci: pin PR labeler to allowlisted 1.3.0 commit#265
svvarom merged 1 commit into
mainfrom
aireinf-208-labeler-allowlist-pin/smuley

Conversation

@svvarom

@svvarom svvarom commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Every "Label PR by type" run fails with startup_failure since #252 merged (e.g. this run). The org's Actions allowlist permits ytanikin/pr-conventional-commits only at b628c5a234cc32513014b7bfdd1e47b532124d98 (1.3.0), but the workflow pins 6ac1cea04190fc076b0e539025501d7e7d241ac1 (1.4.0), so GitHub rejects the workflow before any job starts.

This repins to the allowlisted commit. The only change between 1.3.0 and 1.4.0 is an optional scope-labeling feature we don't use, plus docs — labeling, validation failure on bad titles, and stale-label removal on retitle are identical.

❯ gh api repos/NVIDIA-NeMo/nemo-platform/actions/permissions/selected-actions --jq '.patterns_allowed[] | select(test("ytanikin"))'
ytanikin/pr-conventional-commits@b628c5a234cc32513014b7bfdd1e47b532124d98

Summary by CodeRabbit

  • Chores
    • Updated internal PR workflow automation.

This PR contains only infrastructure updates with no user-facing changes.

Signed-off-by: Swarom Muley <smuley@nvidia.com>
@svvarom
svvarom requested a review from a team as a code owner June 10, 2026 19:21
@svvarom svvarom self-assigned this Jun 10, 2026
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updated the PR labeling workflow to pin a different revision of the ytanikin/pr-conventional-commits action, changing from v1.4.0 to v1.3.0.

Changes

Workflow action pin update

Layer / File(s) Summary
PR conventional commits action pin
.github/workflows/label-pr.yaml
The ytanikin/pr-conventional-commits action reference is pinned to a new commit revision (v1.3.0 instead of v1.4.0).
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed Title accurately describes the main change: pinning the PR labeler workflow to a specific allowlisted commit version (1.3.0).
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch aireinf-208-labeler-allowlist-pin/smuley

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

@svvarom
svvarom enabled auto-merge June 10, 2026 19:30
@github-actions

Copy link
Copy Markdown
Contributor
Suite Lines Covered Line Rate Branch Rate
Unit Tests 19049/25113 75.8% 62.4%
Integration Tests 12058/23885 50.5% 25.7%

@svvarom svvarom added the bug Something isn't working label Jun 10, 2026
@svvarom
svvarom added this pull request to the merge queue Jun 10, 2026
Merged via the queue into main with commit 86732f8 Jun 10, 2026
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants