Skip to content

Add Vishal, Sameek and Anil as codeowners - #5422

Merged
sumitn-xilinx merged 4 commits into
developfrom
new.codeowners
Oct 27, 2026
Merged

sumitn-xilinx merged 4 commits into
developfrom
new.codeowners

Conversation

@asparkhi

Copy link
Copy Markdown
Contributor

Motivation

Add Vishal and Anil as the codeowners.

Copilot AI balanced review requested due to automatic review settings October 22, 2026 11:52

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.

🟡 Changes recommended

The separate wildcard rules cause Vishal’s entry to override Anil’s instead of assigning both as default owners.

1 open issue
What changed in this PR

This PR updates MIGraphX’s default code ownership to designate Anil and Vishal in place of the previous owner.

Changes:

  • Replaces @sumitn-xilinx with entries for @anilmartha and @VishalX.
  • Preserves the documentation-specific ownership rules.
File Description
.github/​CODEOWNERS Updates default owners while retaining documentation ownership.

Review effort: Balanced

Comment thread .github/CODEOWNERS Outdated
Combine @sumitn-xilinx, @asparkhi, @anilmartha, and @VishalX on the same
wildcard rule so all four users receive code-owner review requests. 
Previously, each on a separate line meant only the last rule applied.
@codecov

codecov Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5422      +/-   ##
===========================================
+ Coverage    92.75%   92.94%   +0.19%     
===========================================
  Files          641      667      +26     
  Lines        37716    40034    +2318     
===========================================
+ Hits         34982    37209    +2227     
- Misses        2734     2825      +91     

see 79 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@asparkhi asparkhi changed the title Add Vishal and Anil as codeowners Add Vishal, Sameek and Anil as codeowners Oct 26, 2026
@sumitn-xilinx
sumitn-xilinx merged commit 44c8f17 into develop Oct 27, 2026
31 of 33 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.

3 participants