Text updates (review comments)

This commit is contained in:
Andrew Gross 2018-07-10 07:32:27 -06:00
parent 2121292621
commit 95dce9eee3
1 changed files with 3 additions and 3 deletions

View File

@ -14,9 +14,9 @@ Thank your for contributing to this project! We welcome collaborators and expect
* We strive to follow the [Rust API Guidelines](https://rust-lang-nursery.github.io/api-guidelines/about.html) to maintain consistency and compatibility in our code.
* Commits should be one logical change that still allows all tests to pass. We prefer smaller commits if there could be two levels of logic grouping. The goal is to provide future contributors (including your future self) the reasoning behind your changes and allow them to cherry-pick, patch or port those changes in isolation to other branches or forks.
* If during your PR you reveal a pre-existing bug and know how to fix:
1. If you can isolate the bug, fix in a separate PR.
2. If the fix depends on your other commits, add the fix in a separate commit to the same PR.
* If during your PR you reveal a pre-existing bug and know how to fix it:
1. If you can isolate the bug, fix it in a separate PR.
2. If the fix depends on your other commits, add it in a separate commit to the same PR.
In either case, try to write a regression test that fails because of the bug but passes with your fix.