Skip to content

Commit fde56fa

Browse files
danbevBethGriggs
authored andcommitted
build: make lint-addon-docs run only if needed
Currently, the lint-addon-docs targets recipe will always be run. This commit makes lint-addon-docs a phony target and adds a new target named tools/.doclintstamp what will be an actual file, similar to what the lint-cpp target does. PR-URL: #24993 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
1 parent 3f969d6 commit fde56fa

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

Makefile

+5-1
Original file line numberDiff line numberDiff line change
@@ -1216,10 +1216,14 @@ tools/.cpplintstamp: $(LINT_CPP_FILES)
12161216
@$(PYTHON) tools/check-imports.py
12171217
@touch $@
12181218

1219-
lint-addon-docs: test/addons/.docbuildstamp
1219+
.PHONY: lint-addon-docs
1220+
lint-addon-docs: tools/.doclintstamp
1221+
1222+
tools/.doclintstamp: test/addons/.docbuildstamp
12201223
@echo "Running C++ linter on addon docs..."
12211224
@$(PYTHON) tools/cpplint.py $(CPPLINT_QUIET) --filter=$(ADDON_DOC_LINT_FLAGS) \
12221225
$(LINT_CPP_ADDON_DOC_FILES_GLOB)
1226+
@touch $@
12231227

12241228
cpplint: lint-cpp
12251229
@echo "Please use lint-cpp instead of cpplint"

0 commit comments

Comments
 (0)