Skip to content

chore: simplify CODEOWNERS - #2677

Merged
Rana Singh (ranadeepsingh) merged 1 commit into
microsoft:masterfrom
ranadeepsingh:chore/simplify-codeowners
Aug 25, 2026
Merged

chore: simplify CODEOWNERS#2677
Rana Singh (ranadeepsingh) merged 1 commit into
microsoft:masterfrom
ranadeepsingh:chore/simplify-codeowners

Conversation

@ranadeepsingh

Copy link
Copy Markdown
Collaborator

Related Issues/PRs

None.

What changes are proposed in this pull request?

Replace all path-specific CODEOWNERS entries with one repository-wide rule
assigning @ranadeepsingh and @BrendanWalsh.

How is this patch tested?

  • Confirmed CODEOWNERS contains exactly the requested newline-terminated rule.
  • No automated tests are needed for this metadata-only change.

Does this PR change any dependencies?

  • No.
  • Yes.

Does this PR add a new feature? If so, have you added samples on website?

  • No.
  • Yes.

## Summary
Replace path-specific ownership entries with one repository-wide rule assigning @ranadeepsingh and @BrendanWalsh.

## Prompting Intent
The engineer asked to make CODEOWNERS contain only `*                @ranadeepsingh @BrendanWalsh` and remove every other owner and comment.

## Linked Sources
- User request in this session.

## Rationale
A single global rule makes the requested reviewers responsible for every path and removes narrower overrides that would otherwise supersede it.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 25, 2026 02:30
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@github-actions

Copy link
Copy Markdown

Hey Rana Singh (@ranadeepsingh) 👋!
Thank you so much for contributing to our repository 🙌.
Someone from SynapseML Team will be reviewing this pull request soon.

We use semantic commit messages to streamline the release process.
Before your pull request can be merged, you should make sure your first commit and PR title start with a semantic prefix.
This helps us to create release messages and credit you for your hard work!

Examples of commit messages with semantic prefixes:

  • fix: Fix LightGBM crashes with empty partitions
  • feat: Make HTTP on Spark back-offs configurable
  • docs: Update Spark Serving usage
  • build: Add codecov support
  • perf: improve LightGBM memory usage
  • refactor: make python code generation rely on classes
  • style: Remove nulls from CNTKModel
  • test: Add test coverage for CNTKModel

To test your commit locally, please follow our guild on building from source.
Check out the developer guide for additional guidance on testing your change.

@ranadeepsingh
Rana Singh (ranadeepsingh) merged commit 7f1647a into microsoft:master Aug 25, 2026
11 checks passed

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.

Pull request overview

Simplifies repository ownership configuration by collapsing CODEOWNERS into a single, repo-wide rule for consistent reviewer assignment.

Changes:

  • Remove all path-specific CODEOWNERS entries.
  • Keep a single * rule assigning ownership to @ranadeepsingh and @BrendanWalsh.
Show a summary per file
File Description
CODEOWNERS Consolidates ownership rules into a single repository-wide entry.

Review details

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Suppressed comments (1)

CODEOWNERS:2

  • The file currently has an extra blank line after the single CODEOWNERS rule. This contradicts the PR description claim that it contains exactly the requested newline-terminated rule, and it’s simpler to keep CODEOWNERS to a single non-empty line here.
*                @ranadeepsingh @BrendanWalsh

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

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