diff --git a/.travis.yml b/.travis.yml index 6d2ad6fa..d0cbb081 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,8 +1,16 @@ language: python python: - "3.7" +services: + - "docker" install: - "pip install -r requirements.txt" + - "curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.35.3/install.sh | bash" + - "nvm install" + - "nvm use" + - "npm install -g gulp" + - "npm install" script: - - "pylint --load-plugins pylint_quotes packet/routes packet" + - "gulp lint" + - "docker build -t packet ." diff --git a/README.md b/README.md index df1e2f25..06d4e1a4 100644 --- a/README.md +++ b/README.md @@ -106,7 +106,7 @@ fail blocking you from merging. To make your life easier just run it before maki To run pylint use this command: ```bash -pylint packet +pylint --load-plugins pylint_quotes packet/routes packet ``` All python files should have a top-level docstring explaining the contents of the file and complex functions should diff --git a/gulpfile.js/index.js b/gulpfile.js/index.js index 81238034..d191d579 100644 --- a/gulpfile.js/index.js +++ b/gulpfile.js/index.js @@ -19,4 +19,5 @@ requireDir('./tasks', {recurse: true}); // Default task gulp.task('default', gulp.parallel('css', 'js')); -gulp.task('production', gulp.parallel('css', 'js', 'generate-favicon', 'pylint')); +gulp.task('production', gulp.parallel('css', 'js', 'generate-favicon')); +gulp.task('lint', gulp.parallel('pylint')); diff --git a/gulpfile.js/tasks/pylint.js b/gulpfile.js/tasks/pylint.js index 6a82ab5d..2e93f752 100644 --- a/gulpfile.js/tasks/pylint.js +++ b/gulpfile.js/tasks/pylint.js @@ -2,7 +2,7 @@ const gulp = require('gulp'); const exec = require('child_process').exec; let pylintTask = (cb) => { - exec('pylint packet', function (err, stdout, stderr) { + exec('pylint --load-plugins pylint_quotes packet/routes packet', function (err, stdout, stderr) { console.log(stdout); console.log(stderr); cb(err);