Skip to content

Adding CODEOWNERS File#203

Open
49Indium wants to merge 8 commits into
mainfrom
codeowners
Open

Adding CODEOWNERS File#203
49Indium wants to merge 8 commits into
mainfrom
codeowners

Conversation

@49Indium

Copy link
Copy Markdown
Member

To help contributors find people knowledgeable with each part of the bot and to try to simplify the getting reviews, I've added a CODEOWNERS file. I used a combination of git shortlog -n, git blame and my own knowledge to assign code owners.

I've checked with everyone on this file, but feel free to add your name in spots if you think it's missing. There are a few files without codeowners, but I think that this is ok. These are usually because the main contributor didn't want to be the codeowner, or because the file is very old.

I'll also add documentation for this as well, but that is for the github pages, so will be a different PR.

@jenseni-git

Copy link
Copy Markdown
Member

I'm assuming the current errors are because people were kicked after 2FA?

@jenseni-git jenseni-git left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Consider changing offending tags from @xxxx to @ XXXX. Otherwise, I think it's valuable documentation.

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