Skip to content

Conversation

@LadyKerr
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings January 20, 2026 19:36
Copy link

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

This PR adds a CODEOWNERS file to establish repository maintainership for @LadyKerr and @chrisreddington. However, the file currently uses incorrect syntax that will not function as intended on GitHub.

Changes:

  • Adds a new CODEOWNERS file with maintainer information

@@ -0,0 +1,2 @@
# This repository is maintained by:
Copy link

Copilot AI Jan 20, 2026

Choose a reason for hiding this comment

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

Trailing whitespace detected at the end of this line. Remove the space after the colon.

Suggested change
# This repository is maintained by:
# This repository is maintained by:

Copilot uses AI. Check for mistakes.
@@ -0,0 +1,2 @@
# This repository is maintained by:
* @LadyKerr @chrisreddington
Copy link

Copilot AI Jan 20, 2026

Choose a reason for hiding this comment

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

This CODEOWNERS file syntax is incorrect. GitHub CODEOWNERS files use file patterns followed by owner usernames, not markdown bullet points. The correct format should be a pattern (like * for all files) followed by the owner usernames. For example, to assign all files to these maintainers, use: * @LadyKerr @chrisreddington. The comment on line 1 can remain, but line 2 needs to follow the proper CODEOWNERS syntax to function correctly.

Copilot uses AI. Check for mistakes.
@chrisreddington
Copy link

@LadyKerr – I'm happy for us to leave out code owners for this repos. We've found with other samples that when folks fork it, we'll get tagged for code reviews on their repos. Given this sample is likely to get forked, we should probably avoid that here.

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