diff options
| author | Joseph Richey <joerichey@google.com> | 2018-08-30 13:41:49 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-08-30 13:41:49 -0700 |
| commit | 0f451a722918f39fa07bd9337e4a14ca154b13ae (patch) | |
| tree | 9868ffed8cb74357a06e63b88c56d71b13b415af /CONTRIBUTING.md | |
| parent | 1e1b67dae6c3ae3b5acb5ce377b01b286c3e676b (diff) | |
| parent | 1c9bafdec78b8f238a82314b6d9c566a951486c2 (diff) | |
Merge pull request #107 from google/mod
Use Go Modules and support Go 1.11 building
Diffstat (limited to 'CONTRIBUTING.md')
| -rw-r--r-- | CONTRIBUTING.md | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 08775fd..0b9f168 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -64,8 +64,7 @@ checks you should make sure that in your submission: - If you made any changes to files ending in `.proto`, the corresponding `.pb.go` files should be regenerated with `make gen`. - Any issues found by `make lint` have been addressed. -- If any dependancies have changed, `Gopkg.toml` should be updated and - [`dep ensure`](https://github.com/golang/dep) should be run. +- If any dependancies have changed, run `go mod tidy` and `go mod vendor`. - `make coverage.out` can be used to generate a coverage report for all of the tests, but isn't required for submission (ideally most code would be tested, we are far from that ideal). @@ -75,7 +74,8 @@ Essentially, if you run: make test-setup make all make test-teardown -dep ensure +go mod tidy +go mod vendor ``` and everything succeeds, and no files are changed, you're good to submit. @@ -114,8 +114,11 @@ test filesystem. ### Changing dependencies -fscrypt vendors all of it's dependancies using `dep`. If you add or remove a -dependency, be sure to update `Gopkg.toml` and run `dep ensure`. +fscrypt's dependancies are managed using the [Go 1.11 module system](https://github.com/golang/go/wiki/Modules). +If you add or remove a dependency, be sure to update `go.mod`, `go.sum`, and the +`vendor/` directory by running `go mod tidy` and `go mod vendor`. fscrypt still +vendor's it's dependancies for compatiblity with older users, but this will +probobly be removed once the module system becomes widespread. Also, when adding a dependancy, the license of the package must be compatible with [Apache 2.0](https://www.apache.org/licenses/LICENSE-2.0). See the |