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 notes on how you can help to contributing documentation #842

Merged
merged 2 commits into from
Mar 16, 2015

Conversation

joshsmith
Copy link
Contributor

I wanted to clarify further how someone new to the project could help. This is my first stab at it.

@joaomdmoura
Copy link
Member

👍

@joshsmith
Copy link
Contributor Author

@joaomdmoura no issues with this at all?

@joaomdmoura
Copy link
Member

@joshsmith It sounds great me. There some details that we can add, but it's complementary:

If you want, you can also mention that anyone should feel comfortable to create issues that aim to discuss ideas / problems or implementations.

There are some other specific hints like, squash commits when open a big PR, write tests and always rebase master

@joshsmith
Copy link
Contributor Author

All great ideas. Let me change my PR.

@joshsmith
Copy link
Contributor Author

@joaomdmoura Implemented your suggestions. If you have any more, let me know. Otherwise you can always feel free to open an issue about it.

joshsmith added a commit that referenced this pull request Mar 16, 2015
Add notes on how you can help to contributing documentation
@joshsmith joshsmith merged commit 9058d5f into master Mar 16, 2015
@joshsmith joshsmith deleted the add-contribution-process branch March 16, 2015 06:01
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.

2 participants