p15693087
/
monkey
forked from p15670423/monkey
1
0
Fork 0
Code Issues Pull Requests 1 Projects Releases Wiki Activity

Updated JS lint upper limit (currently 497)

This commit is contained in:
Shay Nehmad 2019-11-07 17:00:14 +02:00
parent fde8ac53e3
commit 0acec95c86
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
.travis.yml
View File

@ -35,20 +35,21 @@ script:
# Display the linter issues # Display the linter issues
- cat flake8_warnings.txt - cat flake8_warnings.txt
# Make sure that we haven't increased the amount of warnings. # Make sure that we haven't increased the amount of warnings.
- WARNINGS_AMOUNT_UPPER_LIMIT=190 - PYTHON_WARNINGS_AMOUNT_UPPER_LIMIT=190
- if [ $(tail -n 1 flake8_warnings.txt) -gt $WARNINGS_AMOUNT_UPPER_LIMIT ]; then echo "Too many warnings! Failing this build. Lower the amount of linter errors in this and try again. " && exit 1; fi - if [ $(tail -n 1 flake8_warnings.txt) -gt $PYTHON_WARNINGS_AMOUNT_UPPER_LIMIT ]; then echo "Too many warnings! Failing this build. Lower the amount of linter errors in this and try again. " && exit 1; fi
- cd monkey # This is our source dir - cd monkey # This is our source dir
- python -m pytest # Have to use `python -m pytest` instead of `pytest` to add "{$builddir}/monkey/monkey" to sys.path. - python -m pytest # Have to use `python -m pytest` instead of `pytest` to add "{$builddir}/monkey/monkey" to sys.path.
# Check JS code. The npm install must happen AFTER the flake8 because the node_modules folder will cause a lot of errors. # Check JS code. The npm install must happen AFTER the flake8 because the node_modules folder will cause a lot of errors.
- cd monkey_island/cc/ui - cd monkey_island/cc/ui
- npm i - npm i
- npm i -g eslint - npm i -g eslint
- cd - - cd -
- cd monkey_island/cc/ui - cd monkey_island/cc/ui
- eslint ./src --quiet - eslint ./src --quiet
- eslint ./src --max-warnings 100 - JS_WARNINGS_AMOUNT_UPPER_LIMIT=497
- eslint ./src --max-warnings $JS_WARNINGS_AMOUNT_UPPER_LIMIT
notifications: notifications:
slack: # Notify to slack slack: # Notify to slack