parent
2441b00262
commit
896aa7c436
|
@ -5,7 +5,7 @@ We love pull requests, they're our favorite.
|
||||||
However, before submitting, please review the following:
|
However, before submitting, please review the following:
|
||||||
|
|
||||||
- Please include tests. Changes with tests will be merged very quickly.
|
- Please include tests. Changes with tests will be merged very quickly.
|
||||||
- Please manually confirm that your changes, and all tests, pass in `bash`, `sh`, and `zsh`. We hope to soon automatically run all these tests, but in the meantime, we have to rely on each contributor doing so.
|
- Please manually confirm that your changes work in `bash`, `sh`/`dash`, `ksh`, and `zsh`. Fast tests do run in these shells, but it's nice to manually verify also.
|
||||||
- Please maintain consistent whitespace - 2-space indentation, trailing newlines in all files, etc.
|
- Please maintain consistent whitespace - 2-space indentation, trailing newlines in all files, etc.
|
||||||
- Any time you make a change to your PR, please rebase freshly on top of master. Nobody likes merge commits.
|
- Any time you make a change to your PR, please rebase freshly on top of master. Nobody likes merge commits.
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue