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

Create CONTRIBUTING #16

Closed
wants to merge 1 commit into from
Closed

Conversation

skinnejj
Copy link

@skinnejj skinnejj commented Oct 3, 2016

This is an initial stub for the requested CONTRIBUTING guidelines issue

This is an initial stub for the requested CONTRIBUTING guidelines issue
@codecov-io
Copy link

codecov-io commented Oct 3, 2016

Current coverage is 96.52% (diff: 100%)

No coverage report found for master at 51d5ebd.

Powered by Codecov. Last update 51d5ebd...c7a91a8

@ankitaggarwal011
Copy link
Owner

Good job @skinnejj. Please add the related information and let me know if you need any help or information. Thanks.

@skinnejj
Copy link
Author

skinnejj commented Oct 3, 2016

Thank you @ankitaggarwal011 ! This was my very first Github contribution and I was really kind of nervous about doing it, so I tried to pick something innocuous that I thought I couldn't mess up too badly. I appreciate your kindness!

@ankitaggarwal011
Copy link
Owner

@skinnejj, Congratulations on starting with the world of open-source contributions. Don't worry about breaking things up as your request will be reviewed before being merged. In addition, read thoroughly about the way GitHub pull-requests and merge works, it'll help you in the long run.

The documentation is one of the most important part of any project and your request is as important as anyone else. Please complete the documentation and let me know if you need any help or information. Any suggestions are most welcome. Thank you for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants