-
Notifications
You must be signed in to change notification settings - Fork 188
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.md #34
Conversation
Great work. I'll let others review this. I don't want to block all reviews on me, personally. It would be better if we distributed the review load across the community. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me as a starting point. Any problems/changes needed can be resolve in another issue.
@aprilllllllll Doesn't look like you have write access. @humphd Can you give @aprilllllllll write access to merge this pull request? |
@aprilllllllll have you accepted the invitation to become a collaborator? I sent it via GitHub. |
@humphd Yes I have accepted the invitation. Can I merge this PR now? |
@aprilllllllll for sure, do it. |
Fixes #27 .
This is clearly a work in progress. I will be working on finding better setup instructions, but for now this will work. I would really like feedback on how to approach issues and pull requests from the community, but what I have mirrors what I saw in other projects.
If there is anything I could add, or information I could use, please let me know. I would like to use docker, as that would simplify things greatly, and I know someone is working on it right now. I will update this as pull requests are merged and the system evolves.