Skip to content

Commit

Permalink
build: don't run lint from test-ci
Browse files Browse the repository at this point in the history
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 <cjihrig@gmail.com>
  Revewied-By: Evan Lucas <evanlucas@me.com>

Reviewed-By: Dummy Reviewer <dummy@reviewer.org>
PR-URL: #11
  • Loading branch information
jbergstroem authored and orangemocha committed Jun 29, 2015
1 parent 9a3d3ae commit dc74f3b
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,9 @@ test-all-http1: test-build
test-all-valgrind: test-build
$(PYTHON) tools/test.py --mode=debug,release --valgrind

test-ci:
$(PYTHON) tools/test.py -ptap --logfile test.tap --mode=release --arch=$(DESTCPU) simple message internet

test-release: test-build
$(PYTHON) tools/test.py --mode=release

Expand Down Expand Up @@ -418,11 +421,6 @@ bench-idle:
sleep 1
./node benchmark/idle_clients.js &

test-ci:
$(PYTHON) tools/test.py -ptap --logfile test.tap --mode=release --arch=$(DESTCPU) simple message internet
$(MAKE) jslint
$(MAKE) cpplint

jslintfix:
PYTHONPATH=tools/closure_linter/ $(PYTHON) tools/closure_linter/closure_linter/fixjsstyle.py --strict --nojsdoc -r lib/ -r src/ --exclude_files lib/punycode.js

Expand Down

0 comments on commit dc74f3b

Please sign in to comment.