Skip to content

Commit dc74f3b

Browse files
jbergstroemorangemocha
authored andcommitted
build: don't run lint from test-ci
Since we will run linting before compiling or testing there's no need to run it as part of the ci testing. Cherry-picked from nodejs/node@8d8a26e Original commit metadata follows: PR-URL: nodejs/node#1965 Reviewed-By: Colin Ihrig <[email protected]> Revewied-By: Evan Lucas <[email protected]> Reviewed-By: Dummy Reviewer <[email protected]> PR-URL: #11
1 parent 9a3d3ae commit dc74f3b

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

Makefile

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,9 @@ test-all-http1: test-build
127127
test-all-valgrind: test-build
128128
$(PYTHON) tools/test.py --mode=debug,release --valgrind
129129

130+
test-ci:
131+
$(PYTHON) tools/test.py -ptap --logfile test.tap --mode=release --arch=$(DESTCPU) simple message internet
132+
130133
test-release: test-build
131134
$(PYTHON) tools/test.py --mode=release
132135

@@ -418,11 +421,6 @@ bench-idle:
418421
sleep 1
419422
./node benchmark/idle_clients.js &
420423

421-
test-ci:
422-
$(PYTHON) tools/test.py -ptap --logfile test.tap --mode=release --arch=$(DESTCPU) simple message internet
423-
$(MAKE) jslint
424-
$(MAKE) cpplint
425-
426424
jslintfix:
427425
PYTHONPATH=tools/closure_linter/ $(PYTHON) tools/closure_linter/closure_linter/fixjsstyle.py --strict --nojsdoc -r lib/ -r src/ --exclude_files lib/punycode.js
428426

0 commit comments

Comments
 (0)