Skip to content

[AURON #2413] fix(ci): adapt labeler.yml to actions/labeler v7#2414

Merged
cxzl25 merged 1 commit into
apache:masterfrom
weiqingy:AURON-2413-labeler-v7
Jul 22, 2026
Merged

[AURON #2413] fix(ci): adapt labeler.yml to actions/labeler v7#2414
cxzl25 merged 1 commit into
apache:masterfrom
weiqingy:AURON-2413-labeler-v7

Conversation

@weiqingy

@weiqingy weiqingy commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #2413.

Rationale for this change

#2410 bumped actions/labeler from v6 to v7, which brought js-yaml ^5.1.0. js-yaml 5.x enforces stricter indentation on multi-line flow sequences and rejects the current .github/labeler.yml with YAMLException: deficient indentation (27:7), so the triage check now fails on PRs that run after that bump.

What changes are included in this PR?

Rewrite the glob lists in .github/labeler.yml from multi-line flow sequences ([ ... ]) to block sequences, for all 17 label entries. The label names and glob patterns are unchanged; only the YAML list style changes.

Verified with js-yaml 5.2.1 (the version labeler v7 resolves): the current file throws deficient indentation (27:7), and the updated file parses cleanly into the same 17 labels and 36 glob patterns.

Are there any user-facing changes?

No. This is a CI configuration fix.

How was this patch tested?

Loaded both the old and new .github/labeler.yml with js-yaml 5.2.1. The old file reproduces the CI error; the new file parses without error and yields identical label-to-glob mappings.

Note

The triage check on this PR is expected to fail until this is merged. The labeler workflow runs on pull_request_target and reads .github/labeler.yml from the base branch (master), not from the PR, so it still sees the current file until the fix lands on master.

actions/labeler was bumped to v7 in apache#2410, which depends on js-yaml ^5.1.0.
js-yaml 5.x is stricter about multi-line flow-sequence indentation and
rejects the current .github/labeler.yml with 'deficient indentation (27:7)',
so the triage check fails on PRs that run after the bump.

Rewrite the 17 glob lists from flow sequences to block sequences. Label
names and glob patterns are unchanged; verified with js-yaml 5.2.1 that the
updated file parses into the same 17 labels and 36 patterns.
Copilot AI review requested due to automatic review settings July 21, 2026 21:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes the GitHub Actions PR labeler (triage) failure after upgrading actions/labeler to v7 by updating .github/labeler.yml to a js-yaml 5.x-compatible YAML style (block sequences instead of multi-line flow sequences), without changing any label names or glob patterns.

Changes:

  • Converted all any-glob-to-any-file glob lists from flow-style ([ ... ]) to block-style (- ...) sequences across all label entries.
  • Kept the existing 17 labels and their 36 glob patterns unchanged while making the YAML parseable under js-yaml 5.x.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@weiqingy

Copy link
Copy Markdown
Contributor Author

The failing triage check on this PR is expected and does not indicate a problem with the change.

The labeler workflow runs on pull_request_target, so it reads .github/labeler.yml from the base branch (master), not from this PR. The error is deficient indentation (27:7), which points at the closing ] on line 27 of the current file on master. This PR rewrites those lists as block sequences and has no ] in the file at all, so that error cannot come from this PR's version. The check is parsing the old config on master, not the fix here.

Once this merges, master carries the block-style config and the next triage run on open PRs parses cleanly. Verified locally with js-yaml 5.2.1 (the version labeler v7 resolves): the current file throws deficient indentation (27:7), and the updated file parses into the same 17 labels and 36 glob patterns.

@cxzl25
cxzl25 merged commit 2889b56 into apache:master Jul 22, 2026
126 of 128 checks passed
@cxzl25

cxzl25 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

@weiqingy Thanks for fixing this issue.

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.

[CI] PR labeler (triage) fails after actions/labeler v7 bump: deficient indentation in labeler.yml

3 participants