Skip to content

Commit ea51d02

Browse files
authored
Include whitespace check in 'make' and 'make ws' (#2630)
Also use 'make ws' in github action. Follow up to #2603
1 parent 78bc98b commit ea51d02

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

.github/workflows/push.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ jobs:
114114
# Exit with status code 1 if there are differences (i.e. unformatted files)
115115
git diff --exit-code
116116
- name: Run whitespace check
117-
run: python tools/validate_whitespace.py
117+
run: make ws
118118

119119
validate-bundle-schema:
120120
needs: cleanups

Makefile

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
default: vendor fmt lint tidy
1+
default: vendor fmt lint tidy ws
22

33
PACKAGES=./acceptance/... ./libs/... ./internal/... ./cmd/... ./bundle/... .
44

@@ -20,6 +20,9 @@ fmt:
2020
ruff format -qn
2121
golangci-lint fmt
2222

23+
ws:
24+
./tools/validate_whitespace.py
25+
2326
test:
2427
${GOTESTSUM_CMD} -- ${PACKAGES}
2528

@@ -65,4 +68,4 @@ generate:
6568
[ ! -f tagging.py ] || mv tagging.py internal/genkit/tagging.py
6669
[ ! -f .github/workflows/next-changelog.yml ] || rm .github/workflows/next-changelog.yml
6770

68-
.PHONY: lint tidy lintcheck fmt test cover showcover build snapshot vendor schema integration integration-short acc-cover acc-showcover docs
71+
.PHONY: lint tidy lintcheck fmt test cover showcover build snapshot vendor schema integration integration-short acc-cover acc-showcover docs ws

0 commit comments

Comments
 (0)