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.md #34

Merged
merged 3 commits into from
Nov 6, 2019
Merged

Add CONTRIBUTING.md #34

merged 3 commits into from
Nov 6, 2019

Conversation

drwm-base
Copy link
Contributor

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.

@drwm-base drwm-base requested a review from humphd November 5, 2019 03:56
@humphd humphd removed their request for review November 5, 2019 18:50
@humphd
Copy link
Contributor

humphd commented Nov 5, 2019

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.

Copy link
Contributor

@Immutablevoid Immutablevoid left a 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.

@Immutablevoid
Copy link
Contributor

Immutablevoid commented Nov 6, 2019

@aprilllllllll Doesn't look like you have write access. @humphd Can you give @aprilllllllll write access to merge this pull request?

@humphd
Copy link
Contributor

humphd commented Nov 6, 2019

@aprilllllllll have you accepted the invitation to become a collaborator? I sent it via GitHub.

@aprilllllllll
Copy link
Contributor

@humphd Yes I have accepted the invitation. Can I merge this PR now?

@humphd
Copy link
Contributor

humphd commented Nov 6, 2019

@aprilllllllll for sure, do it.

@Immutablevoid Immutablevoid merged commit e28043a into Seneca-CDOT:master Nov 6, 2019
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.

Add a CONTRIBUTING.md file for environment setup and tracking
5 participants