Skip to content

CODEOWNERS: Add DanielKellerM and fix path typos#101

Merged
DanielKellerM merged 1 commit into
pulp-platform:masterfrom
DanielKellerM:chore/codeowners
May 18, 2026
Merged

CODEOWNERS: Add DanielKellerM and fix path typos#101
DanielKellerM merged 1 commit into
pulp-platform:masterfrom
DanielKellerM:chore/codeowners

Conversation

@DanielKellerM
Copy link
Copy Markdown
Collaborator

@DanielKellerM DanielKellerM commented May 16, 2026

  • Add @DanielKellerM as a global code owner alongside @thommythomaso.
  • Fix src/frontendssrc/frontend
  • Drop the src/systems entry, dir no longer exists

…entry

Add @DanielKellerM as a global owner alongside @thommythomaso.

Fix path-based rules: 'src/frontends' (plural) does not exist, the real
directory is 'src/frontend' (singular), so @micprog was never actually
auto-assigned. Drop the 'src/systems' entry; no such directory exists.
Copilot AI review requested due to automatic review settings May 16, 2026 10:13
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CODEOWNERS file to add a new global code owner and corrects path entries so auto-assignment actually works.

Changes:

  • Adds @DanielKellerM as a global code owner alongside @thommythomaso.
  • Fixes src/frontendssrc/frontend (matches the actual directory name).
  • Removes the src/systems entry since no such directory exists in this repo.

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

@DanielKellerM DanielKellerM deleted the chore/codeowners branch May 17, 2026 17:41
@DanielKellerM DanielKellerM restored the chore/codeowners branch May 17, 2026 17:45
@DanielKellerM DanielKellerM reopened this May 17, 2026
@DanielKellerM DanielKellerM merged commit 0512611 into pulp-platform:master May 18, 2026
32 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