Skip to content

Update CODEOWNERS to remove secexp#4446

Open
shati-patel wants to merge 1 commit into
mainfrom
shati-patel-patch-1
Open

Update CODEOWNERS to remove secexp#4446
shati-patel wants to merge 1 commit into
mainfrom
shati-patel-patch-1

Conversation

@shati-patel

Copy link
Copy Markdown
Contributor

I think these have all been handed over to CodeQL, so @github/code-scanning-secexp-reviewers probably don't need to be codeowners any more?

@shati-patel shati-patel requested a review from a team as a code owner July 8, 2026 13:58
Copilot AI review requested due to automatic review settings July 8, 2026 13:58

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

This PR simplifies repository ownership rules by removing @github/code-scanning-secexp-reviewers as CODEOWNERS for several subdirectories, leaving @github/codeql-vscode-reviewers as the sole codeowner across the tree.

Changes:

  • Removed directory-specific CODEOWNERS entries for variant-analysis, databases, method-modeling, model-editor, and queries-panel that previously required @github/code-scanning-secexp-reviewers.
  • Kept the global ownership rule **/* @github/codeql-vscode-reviewers as the only remaining rule.
Show a summary per file
File Description
CODEOWNERS Removes secexp reviewer ownership entries, leaving a single global CodeQL VS Code reviewers rule.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

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