Skip to content

fix(ci): Reduce pr risk label check to warning#9411

Open
andrew-eldridge wants to merge 1 commit into
mainfrom
aeldridge/prValidationFix
Open

fix(ci): Reduce pr risk label check to warning#9411
andrew-eldridge wants to merge 1 commit into
mainfrom
aeldridge/prValidationFix

Conversation

@andrew-eldridge

@andrew-eldridge andrew-eldridge commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

The AI PR validation workflow regularly has false positive errors on risk level mismatch for PRs, forcing PR author to change risk level to an incorrect value to unblock the PR. Makes a mismatch between agent estimation and true value a warning rather than error to avoid blocking valid PRs.

Impact of Change

  • Users: N/A
  • Developers: Reduces false positive PR validation failures.
  • System: N/A

Test Plan

  • Unit tests added/updated
  • E2E tests added/updated
  • Manual testing completed
  • Tested in:

Contributors

@andrew-eldridge

@andrew-eldridge andrew-eldridge added the risk:low Low risk change with minimal impact label Jul 16, 2026

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@andrew-eldridge andrew-eldridge added risk:low Low risk change with minimal impact and removed risk:low Low risk change with minimal impact labels Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:low Low risk change with minimal impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants