Skip to content

chore: add instructions to update CODEOWNERS file#16566

Merged
ohmayr merged 1 commit intomainfrom
update-documentation-for-codeowners
Apr 6, 2026
Merged

chore: add instructions to update CODEOWNERS file#16566
ohmayr merged 1 commit intomainfrom
update-documentation-for-codeowners

Conversation

@ohmayr
Copy link
Copy Markdown
Contributor

@ohmayr ohmayr commented Apr 6, 2026

add instructions to update CODEOWNERS file

@ohmayr ohmayr requested review from a team as code owners April 6, 2026 20:25
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a set of instructions for modifying the .github/CODEOWNERS file and reorders the package entries alphabetically. Feedback was provided to explicitly document the alphabetical sorting requirement within the new instructions to ensure future consistency.

# This file controls who is tagged for review for any given pull request.
#
# To add a new package or team:
# 1. Add the directory path below the default catch-all rule (`*`).
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.

medium

To ensure the list of package owners remains sorted, please update this instruction to explicitly mention that new entries should maintain alphabetical order. This aligns with the reordering change in this PR and will help maintain the file's structure.

# 1. Add the directory path below the default catch-all rule (*), keeping the list sorted alphabetically.

@ohmayr ohmayr merged commit 27af653 into main Apr 6, 2026
31 checks passed
@ohmayr ohmayr deleted the update-documentation-for-codeowners branch April 6, 2026 21:16
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