Skip to content

I18N-1387 Support skipping i18n checks via GitHub PR label - #494

Merged
DarKhaos merged 1 commit into
masterfrom
I18N-1387
Jul 21, 2026
Merged

I18N-1387 Support skipping i18n checks via GitHub PR label#494
DarKhaos merged 1 commit into
masterfrom
I18N-1387

Conversation

@DarKhaos

@DarKhaos DarKhaos commented Jul 20, 2026

Copy link
Copy Markdown

Summary
Adds support for skipping i18n checks using a GitHub PR label in addition to the existing comment-based mechanism. PRs can now be labeled with skip-i18n-checks to bypass i18n validation checks without requiring a comment.

Changes

  • Added --skip-i18n-checks-label parameter to GithubPRInfoCommand (defaults to skip-i18n-checks)
  • Modified check logic to support both label-based and comment-based skip mechanisms
  • Only adds reaction to comment when checks are skipped via comment (not via label)
  • Updated commit status message to reflect both skip methods
  • Added comprehensive test coverage for label-only and combined label+comment scenarios

Test Plan

  • Unit tests added for label-based skip functionality
  • Tests verify correct behavior when both label and comment are present
  • Tests confirm reaction is only added to comments, not when label is used

@DarKhaos
DarKhaos force-pushed the I18N-1387 branch 2 times, most recently from 2a2761d to 26dc97b Compare July 20, 2026 22:45
Summary
Adds support for skipping i18n checks using a GitHub PR label in addition to the existing comment-based mechanism. PRs can now be labeled with skip-i18n-checks to bypass i18n validation checks without requiring a comment.

Changes
- Added --skip-i18n-checks-label parameter to GithubPRInfoCommand (defaults to skip-i18n-checks)
- Modified check logic to support both label-based and comment-based skip mechanisms
- Only adds reaction to comment when checks are skipped via comment (not via label)
- Updated commit status message to reflect both skip methods
- Added comprehensive test coverage for label-only and combined label+comment scenarios

Test Plan
- Unit tests added for label-based skip functionality
- Tests verify correct behavior when both label and comment are present
- Tests confirm reaction is only added to comments, not when label is used
@DarKhaos
DarKhaos marked this pull request as ready for review July 20, 2026 23:47
@DarKhaos
DarKhaos requested a review from a team as a code owner July 20, 2026 23:47
@DarKhaos
DarKhaos merged commit 278856d into master Jul 21, 2026
2 checks passed
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.

2 participants