docs/Maintainer-Guidelines: don't merge failing tests.
This commit is contained in:
parent
00075c228c
commit
3245b91900
@ -85,6 +85,11 @@ If the formula uses a repository, then the `url` parameter should have a
|
||||
tag or revision. `url`s have versions and are stable (not yet
|
||||
implemented!).
|
||||
|
||||
Don't merge any formula updates with failing `brew test`s. If a `test do` block
|
||||
is failing it needs to be fixed. This may involve replacing more involved tests
|
||||
with those that are more reliable. This is fine: false positives are better than
|
||||
false negatives as we don't want to teach maintainers to merge red PRs.
|
||||
|
||||
## Common “gotchas”
|
||||
1. [Ensure you have set your username and email address
|
||||
properly](https://help.github.com/articles/setting-your-email-in-git/)
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user