makefile: Raname targets update* to fix* to distinquish from update_dep

Signed-off-by: Marek Siarkowicz <siarkowicz@google.com>
This commit is contained in:
Marek Siarkowicz 2022-09-20 13:53:16 +02:00
parent 5bfda80836
commit bea478266e
3 changed files with 7 additions and 10 deletions

View File

@ -13,4 +13,4 @@ jobs:
with:
version: v1.49.0
- run: make verify
- run: make update
- run: make fix

View File

@ -67,8 +67,8 @@ See the [style doc](https://github.com/golang/go/wiki/CodeReviewComments) for de
Please ensure that your change passes static analysis (requires [golangci-lint](https://golangci-lint.run/usage/install/)):
- `make verify` to verify if all checks pass.
- `make verify-*` to verify a single check, for example `make verify-bom` to verify if bill-of-materials.json file is up-to-date.
- `make update` to fix all checks.
- `make update-*` to fix a single checks, for example `make update-bom` to update bill-of-materials.json.
- `make fix` to fix all checks.
- `make fix-*` to fix a single checks, for example `make fix-bom` to update bill-of-materials.json.
Please ensure that your change passes tests.
- `make test-unit` to run unit tests.

View File

@ -30,7 +30,8 @@ test-e2e-release: build
# Static analysis
verify: verify-gofmt verify-bom verify-lint verify-dep verify-shellcheck verify-goword verify-govet verify-license-header verify-receiver-name verify-mod-tidy verify-shellcheck verify-shellws verify-proto-annotations
update: update-bom update-lint update-fix
fix: fix-bom fix-lint
./scripts/fix.sh
.PHONY: verify-gofmt
verify-gofmt:
@ -41,7 +42,7 @@ verify-bom:
PASSES="bom" ./scripts/test.sh
.PHONY: update-bom
update-bom:
fix-bom:
./scripts/updatebom.sh
.PHONY: verify-dep
@ -53,13 +54,9 @@ verify-lint:
golangci-lint run
.PHONY: update-lint
update-lint:
fix-lint:
golangci-lint run --fix
.PHONY: update-fix
update-fix:
./scripts/fix.sh
.PHONY: verify-shellcheck
verify-shellcheck:
PASSES="shellcheck" ./scripts/test.sh