Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add contribution instructions, code of conduct, and codeowners #122

Merged
merged 5 commits into from
Oct 26, 2022

Conversation

bensheldon
Copy link
Contributor

@bensheldon bensheldon commented Oct 13, 2022

Connects to #109.

@bensheldon bensheldon changed the title Create CONTRIBUTING.md Add contribution instructions, code of conduct, and codeowners Oct 13, 2022

This repository, `rubocop-github` is part of a broader RuboCop ecosystem.

If the Cop you would like to propose is **generally applicable outside of GitHub**:
Copy link
Member

@issyl0 issyl0 Oct 18, 2022

Choose a reason for hiding this comment

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

Should we be stronger here about only allowing contributions from GitHub staff? Maybe something like "we only accept new cop contributions if there's evidence they're applicable to and useful for GitHub's codebase". Not sure how external folks will provide that evidence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good flag! I'm willing to defer that until we see it. I'm not too worried that we'll be overwhelmed by contributions 😆

Copy link
Contributor

@sampart sampart left a comment

Choose a reason for hiding this comment

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

Thanks so much for moving this forward ❤️ Just a quick grammar suggestion.

CONTRIBUTING.md Outdated Show resolved Hide resolved
@bensheldon bensheldon merged commit 1567efb into main Oct 26, 2022
@bensheldon bensheldon deleted the bensheldon/contributing-instructions branch October 26, 2022 15:05
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