Skip to content

Verify roles: all is valid and compiles correctly#14900

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/regression-test-roles-all-error
Closed

Verify roles: all is valid and compiles correctly#14900
Copilot wants to merge 1 commit intomainfrom
copilot/regression-test-roles-all-error

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Investigation of reported issue that roles: all errors during compilation.

Findings

No bug exists. The compiler correctly accepts roles: all:

---
description: Test workflow
roles: all
on:
  issue_comment:
    types: [created]
---

Compiles successfully with exit code 0.

Verification

  • Documentation confirms roles: all is valid syntax to allow any authenticated user
  • Schema correctly defines roles as oneOf: string "all" OR array of role names
  • Existing tests (TestBotsWithRolesAll, etc.) pass and validate this behavior

Note

roles: [all] (array containing "all") correctly fails validation—"all" is not a valid role name in array format. This is expected behavior per the schema.

No code changes required.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Add regression test for roles: all compiler error Verify roles: all is valid and compiles correctly Feb 11, 2026
Copilot AI requested a review from eaftan February 11, 2026 06:25
@github-actions
Copy link
Contributor

🔍 PR Triage Results

Category: chore | Risk: low | Priority: 20/100

Scores Breakdown

  • Impact: 5/50 - No code changes - investigation confirming existing behavior works correctly
  • Urgency: 5/30 - Documentation/verification PR, draft status, no urgency
  • Quality: 10/20 - Good description explaining findings (+5), no tests needed for doc, draft status

📋 Recommended Action: close

Rationale: No code changes required. Investigation concluded that no bug exists - roles: all works correctly as documented.

Recommendation: This PR should be closed rather than merged since it contains no code changes. The investigation findings can be referenced in comments or documentation if needed, but there's nothing to merge.


Triaged by PR Triage Agent on 2026-02-11

AI generated by PR Triage Agent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants