mirror of https://github.com/zulip/zulip.git
travis: Move i18n and capitalization linters to frontend.
Since these usually result from changes to HTML templates and other frontend-side things, it seems better to group them with the frontend. [Tweaked by gnprice in whitespace and comments.]
This commit is contained in:
parent
6ef55145f5
commit
b087886d1c
|
@ -36,9 +36,6 @@ run ./tools/clean-repo
|
|||
|
||||
# travis/backend
|
||||
run ./tools/lint $FORCEARG
|
||||
run ./manage.py makemessages --locale en
|
||||
run env PYTHONWARNINGS=ignore ./tools/check-capitalization --no-generate
|
||||
run env PYTHONWARNINGS=ignore ./tools/check-frontend-i18n --no-generate
|
||||
run ./tools/test-tools
|
||||
run ./tools/test-backend $FORCEARG
|
||||
run ./tools/run-mypy
|
||||
|
@ -56,6 +53,9 @@ run ./tools/test-api
|
|||
|
||||
# travis/frontend
|
||||
run ./tools/test-js-with-node
|
||||
run ./manage.py makemessages --locale en
|
||||
run env PYTHONWARNINGS=ignore ./tools/check-capitalization --no-generate
|
||||
run env PYTHONWARNINGS=ignore ./tools/check-frontend-i18n --no-generate
|
||||
run ./tools/test-js-with-casper $FORCEARG
|
||||
|
||||
printf '\n\e[32mAll OK!\e[0m\n'
|
||||
|
|
|
@ -6,10 +6,6 @@ set -e
|
|||
set -x
|
||||
|
||||
./tools/lint --backend
|
||||
./manage.py makemessages --locale en
|
||||
PYTHONWARNINGS=ignore ./tools/check-capitalization --no-generate
|
||||
PYTHONWARNINGS=ignore ./tools/check-frontend-i18n --no-generate
|
||||
|
||||
./tools/test-tools
|
||||
./tools/test-backend --coverage
|
||||
|
||||
|
|
|
@ -6,7 +6,16 @@ set -e
|
|||
set -x
|
||||
|
||||
./tools/lint --frontend
|
||||
|
||||
# Run the node tests first, since they're fast and deterministic
|
||||
./tools/test-js-with-node --coverage
|
||||
|
||||
# Check capitalization of strings
|
||||
./manage.py makemessages --locale en
|
||||
PYTHONWARNINGS=ignore ./tools/check-capitalization --no-generate
|
||||
PYTHONWARNINGS=ignore ./tools/check-frontend-i18n --no-generate
|
||||
|
||||
# Run the slower Casper tests last
|
||||
./tools/test-js-with-casper
|
||||
|
||||
# NB: Everything here should be in `tools/test-all`. If there's a
|
||||
|
|
Loading…
Reference in New Issue