Skip to content

Commit a635193

Browse files
ci: skip verification of vendored paths
This patch updates configuration of the github lint action along with markdown and golangci-lint. Signed-off-by: Niraj Yadav <niryadav@redhat.com>
1 parent ce02d52 commit a635193

3 files changed

Lines changed: 9 additions & 4 deletions

File tree

.github/workflows/lint.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@ name: lint
22
on:
33
pull_request:
44
paths-ignore:
5-
- 'vendor/*'
5+
- 'vendor/**'
6+
- 'api/vendor/**'
67
branches:
78
- main
89
- release-*
@@ -35,7 +36,7 @@ jobs:
3536
run: |
3637
codespell \
3738
--ignore-words-list="AfterAll,NotIn,notin,immediatedly" \
38-
--skip=".git,*.sum,vendor"
39+
--skip=".git,*.sum,vendor,api/vendor"
3940
4041
golangci:
4142
name: golangci-lint
@@ -89,6 +90,7 @@ jobs:
8990
run: |
9091
find . -type f -name "*.sh" \
9192
! -path "./vendor/*" \
93+
! -path "./api/vendor/*" \
9294
-exec shellcheck {} +
9395
9496
modcheck:

.golangci.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ issues:
66
# don't skip warning about doc comments
77
# don't exclude the default set of lint
88
exclude-use-default: false
9+
exclude-dirs:
10+
- vendor
11+
- api/vendor
912
# restore some of the defaults
1013
# (fill in the rest as needed)
1114
exclude-rules:

Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,11 +135,11 @@ test-e2e:
135135
MARKDOWNLINT_IMAGE = docker.io/davidanson/markdownlint-cli2:v0.17.1
136136
.PHONY: markdownlint
137137
markdownlint:
138-
@$(CONTAINER_TOOL) run --platform linux/amd64 -v .\:/workdir\:z $(MARKDOWNLINT_IMAGE) markdownlint-cli2 "**.md" "#vendor/**" --config .markdownlint.yaml
138+
@$(CONTAINER_TOOL) run --platform linux/amd64 -v .\:/workdir\:z $(MARKDOWNLINT_IMAGE) markdownlint-cli2 "**.md" "#vendor/**" "#api/vendor/**" --config .markdownlint.yaml
139139

140140
.PHONY: markdownlint-fix
141141
markdownlint-fix:
142-
@$(CONTAINER_TOOL) run --platform linux/amd64 -v .\:/workdir\:z $(MARKDOWNLINT_IMAGE) markdownlint-cli2 "**.md" "#vendor/**" --config .markdownlint.yaml --fix
142+
@$(CONTAINER_TOOL) run --platform linux/amd64 -v .\:/workdir\:z $(MARKDOWNLINT_IMAGE) markdownlint-cli2 "**.md" "#vendor/**" "#api/vendor/**" --config .markdownlint.yaml --fix
143143

144144

145145
.PHONY: golangci-lint-fix

0 commit comments

Comments
 (0)