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 contributing and developers guide documentation #155

Merged
merged 1 commit into from
Jan 26, 2018

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Jan 12, 2018

This adds a contributing page and basic developer's guide to the sphinx documentation. The contributing file actually lives in the root of the repository so that github can find it and suggest contributors read it before submitting pull requests.

This also includes a fix to one of the new helper functions so sphinx can build it.

  • Fully documented

@djhoese djhoese self-assigned this Jan 12, 2018
@djhoese djhoese requested a review from mraspaud January 12, 2018 21:06
@CLAassistant
Copy link

CLAassistant commented Jan 12, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coveralls
Copy link

coveralls commented Jan 12, 2018

Coverage Status

Coverage remained the same at 57.57% when pulling fdf5133 on doc-contributing into 58d2ec8 on develop.

@mraspaud mraspaud added this to the v0.9 milestone Jan 19, 2018
Copy link
Member

@mraspaud mraspaud left a comment

Choose a reason for hiding this comment

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

LGTM

@djhoese djhoese merged commit 2e86456 into develop Jan 26, 2018
@djhoese djhoese deleted the doc-contributing branch January 26, 2018 22:16
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

4 participants