Skip to content

Commit

Permalink
Merge pull request #837 from urfave/codeowners
Browse files Browse the repository at this point in the history
Use codeowners instead of maintainers
  • Loading branch information
asahasrabuddhe committed Aug 4, 2019
2 parents 7745000 + 1327f58 commit d09efb5
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 9 deletions.
4 changes: 4 additions & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
@@ -0,0 +1,4 @@
# See https://help.github.com/articles/about-codeowners/
# for more info about CODEOWNERS file

* @urfave/cli
9 changes: 4 additions & 5 deletions CONTRIBUTING.md
@@ -1,11 +1,10 @@
## Contributing

**NOTE**: the primary maintainer(s) may be found in
[./MAINTAINERS.md](./MAINTAINERS.md).
Use @urfave/cli to ping the maintainers.

Feel free to put up a pull request to fix a bug or maybe add a feature. I will
Feel free to put up a pull request to fix a bug or maybe add a feature. We will
give it a code review and make sure that it does not break backwards
compatibility. If I or any other collaborators agree that it is in line with
compatibility. If collaborators agree that it is in line with
the vision of the project, we will work with you to get the code into
a mergeable state and merge it into the master branch.

Expand All @@ -15,5 +14,5 @@ to merge others pull requests. It is very important that new code does not
break existing code, so be careful about what code you do choose to merge.

If you feel like you have contributed to the project but have not yet been added
as a collaborator, we probably forgot to add you :sweat_smile:. Please open an
as a collaborator, we probably forgot to add you :sweat_smile:. Please open an
issue!
4 changes: 0 additions & 4 deletions MAINTAINERS.md

This file was deleted.

0 comments on commit d09efb5

Please sign in to comment.