Merge pull request #18007 from MadhavJivrajani/refactor-fix

split out subroutine from fix.sh to makefile target
etcd-io · May 14, 2024 · 22c40e1 · 22c40e1
2 parents 7b9013d + b915323
commit 22c40e1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/Makefile b/Makefile
@@ -58,7 +58,7 @@ verify: verify-gofmt verify-bom verify-lint verify-dep verify-shellcheck verify-
 	verify-govet-shadow verify-markdown-marker verify-go-versions
 
 .PHONY: fix
-fix: fix-bom fix-lint fix-yamllint
+fix: fix-bom fix-lint fix-yamllint sync-toolchain-directive
 	./scripts/fix.sh
 
 .PHONY: verify-gofmt

diff --git a/scripts/fix.sh b/scripts/fix.sh
@@ -36,7 +36,4 @@ run_for_modules run ${GO_CMD} fmt || exit 2
 run_for_module tests bom_fix || exit 2
 bash_ws_fix || exit 2
 
-log_callout "Syncing go toolchain directives"
-run ./scripts/sync_go_toolchain_directive.sh || exit 2
-
 log_success -e "\\nSUCCESS: etcd code is fixed :)"