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

Simplify a great deal the CONTRIBUTING file #40

Merged
merged 5 commits into from
Feb 15, 2016

Conversation

astorije
Copy link
Member

It was brought to my attention that, if CONTRIBUTING files are often ignored, way too long CONTRIBUTING files are pretty much always ignored.

Prior to that PR, there were lots of unnecessary, lengthy details that neither casual or involved contributors do not care about.
Casual contributors fix comments, super minor bugs or documentation pieces, all of which we should simply be thankful of.
Involved contributors should rightfully enquire more before starting coding, which this PR details sufficiently (mainly open a PR or discuss on IRC first). Very involved contributors will read our maintainers' corner.

Travis CI automatically checks for this now
Instead, link to the maintainers corner that contain this information
This information is now available in the maintainers corner
@astorije astorije added Type: Documentation Lack of documentation, improvement suggestion, or PRs that address these. second review needed labels Feb 14, 2016
@xPaw
Copy link
Member

xPaw commented Feb 14, 2016

👍

@astorije
Copy link
Member Author

I will actually wait for a second 👍 from before merging this.

@AlMcKinlay
Copy link
Member

👍

AlMcKinlay added a commit that referenced this pull request Feb 15, 2016
Simplify a great deal the CONTRIBUTING file
@AlMcKinlay AlMcKinlay merged commit e3daa9e into master Feb 15, 2016
@AlMcKinlay AlMcKinlay deleted the astorije/simpler-contribution-file branch February 15, 2016 08:33
@astorije astorije assigned xPaw and unassigned xPaw Feb 25, 2016
@astorije astorije added this to the 1.1.0 milestone Apr 1, 2017
@xPaw xPaw removed their assignment Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Lack of documentation, improvement suggestion, or PRs that address these.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants