Remove dep from contribution guidelines (#7180)
This commit is contained in:
parent
fcb038d352
commit
4162599081
|
@ -30,30 +30,24 @@ and the accumulator can be found in the GoDoc:
|
||||||
|
|
||||||
**Adding a dependency:**
|
**Adding a dependency:**
|
||||||
|
|
||||||
Assuming you can already build the project, run these in the telegraf directory:
|
Telegraf uses Go modules. Assuming you can already build the project, run this in the telegraf directory:
|
||||||
|
|
||||||
1. `dep ensure -vendor-only`
|
1. `go get github.com/[dependency]/[new-package]`
|
||||||
2. `dep ensure -add github.com/[dependency]/[new-package]`
|
|
||||||
|
|
||||||
**Unit Tests:**
|
**Unit Tests:**
|
||||||
|
|
||||||
Before opening a pull request you should run the linter checks and
|
Before opening a pull request you should run the linter checks and
|
||||||
the short tests.
|
the short tests.
|
||||||
|
|
||||||
**Run static analysis:**
|
|
||||||
|
|
||||||
```
|
```
|
||||||
make check
|
make check
|
||||||
```
|
|
||||||
|
|
||||||
**Run short tests:**
|
|
||||||
|
|
||||||
```
|
|
||||||
make test
|
make test
|
||||||
```
|
```
|
||||||
|
|
||||||
**Execute integration tests:**
|
**Execute integration tests:**
|
||||||
|
|
||||||
|
(Optional)
|
||||||
|
|
||||||
Running the integration tests requires several docker containers to be
|
Running the integration tests requires several docker containers to be
|
||||||
running. You can start the containers with:
|
running. You can start the containers with:
|
||||||
```
|
```
|
||||||
|
|
Loading…
Reference in New Issue