|
|
|
@ -57,6 +57,17 @@ After running for a while, the command should print
|
|
|
|
|
```
|
|
|
|
|
ALL TESTS PASSED
|
|
|
|
|
```
|
|
|
|
|
## Vendoring
|
|
|
|
|
|
|
|
|
|
We keep a cached copy of dependencies within the `vendor/` directory,
|
|
|
|
|
managing updates via [govendor](http://github.com/kardianos/govendor).
|
|
|
|
|
|
|
|
|
|
Pull requests should only include `vendor/` updates if they are
|
|
|
|
|
part of the same change, be it a bugfix or a feature addition.
|
|
|
|
|
|
|
|
|
|
The `vendor/` update needs to be justified as part of the PR description,
|
|
|
|
|
and must be verified by the reviewers and/or merger to always reference
|
|
|
|
|
an existing upstream commit.
|
|
|
|
|
|
|
|
|
|
## Code review
|
|
|
|
|
|
|
|
|
|