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

Moved Tests up in Contribute.md #2655

Merged
merged 2 commits into from
May 11, 2016
Merged

Conversation

RichardLitt
Copy link
Member

This should be more prominent. We also need a much better description of how to run tests - rigt now, we are dependant on Travis for the majority of testing, I believe. If there was a clear guide here on how to run the appropriate tests for each kind of PR, that would be fantastic.

We also really ought to have a section asking people to add all of the tests that they can think of.

License: MIT
Signed-off-by: Richard Littauer richard.littauer@gmail.com

This should be more prominent. We also need a much better description of how to run tests - rigt now, we are dependant on Travis for the majority of testing, I believe. If there was a clear guide here on how to run the appropriate tests for each kind of PR, that would be fantastic.

We also really ought to have a section asking people to add all of the tests that they can think of.

License: MIT
Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
@RichardLitt RichardLitt added the topic/docs-ipfs Topic docs-ipfs label May 10, 2016
License: MIT
Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
@RichardLitt RichardLitt force-pushed the feature/contribute-tests-section branch from 14e9326 to b620817 Compare May 10, 2016 22:07
@whyrusleeping whyrusleeping merged commit 732bd78 into master May 11, 2016
@whyrusleeping whyrusleeping deleted the feature/contribute-tests-section branch May 11, 2016 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/docs-ipfs Topic docs-ipfs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants