Skip to content

fix: add e2e-spec pattern to CODEOWNERS test file exclusions#27926

Merged
anikdhabal merged 2 commits intomainfrom
codeowner
Feb 13, 2026
Merged

fix: add e2e-spec pattern to CODEOWNERS test file exclusions#27926
anikdhabal merged 2 commits intomainfrom
codeowner

Conversation

@anikdhabal
Copy link
Copy Markdown
Contributor

@anikdhabal anikdhabal commented Feb 13, 2026 •
edited by devin-ai-integration bot
Loading

Summary by cubic

Expanded CODEOWNERS test exclusions to include e2e-spec and test-suite files so they no longer request foundation/consumer approval.

Written for commit c36b6e6. Summary will update on new commits.

Open with Devin

@graphite-app graphite-app bot added the core area: core, team members only label Feb 13, 2026
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Feb 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@anikdhabal anikdhabal merged commit 6386b2e into main Feb 13, 2026
45 checks passed
@anikdhabal anikdhabal deleted the codeowner branch February 13, 2026 09:48
devin-ai-integration[bot]
devin-ai-integration bot reviewed Feb 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@paragon-review
Copy link
Copy Markdown

paragon-review bot commented Feb 13, 2026

Paragon: tests updated

1 updated test generated for this PR.

Updated Tests

  • CODEOWNERS file structure and exclusion pattern tests — Comprehensive tests for the CODEOWNERS file covering: (1) file structure validation (existence, non-empty, valid team handles), (2) all ownership rules for Foundation/Consumer teams across 39 patterns, (3) all 5 test exclusion patterns including the new **/.e2e-spec. pattern from PR fix: add e2e-spec pattern to CODEOWNERS test file exclusions #27926, (4) pattern matching tests against real repo file paths (e.g. organizations-organizations.controller.e2e-spec.ts from apps/api/v2), (5) edge cases (deeply nested paths, various extensions, root-level files, non-test files in test directories), (6) completeness checks (no duplicates, alignment with vitest.config.mts and jest-e2e.ts conventions), (7) regex converter unit tests for the CODEOWNERS glob-to-regex helper

Accept Changes Open in Paragon

Details

Updated Tests

  • CODEOWNERS file structure and exclusion pattern tests (unit)

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

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

@hariombalhara hariombalhara hariombalhara approved these changes

Assignees

No one assigned

Labels

core area: core, team members only ready-for-e2e size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anikdhabal @hariombalhara