Merge pull request #2384 from betagouv/move-linters-to-rake-task

Move linters to a Rake task

.circleci/config.yml

@@ -90,20 +90,8 @@ jobs:
       - *yarn_restore_cache
       - *yarn_install
       - run:
-          name: Run eslint
+          name: Run linters
-          command: yarn lint:js
+          command: bundle exec rake lint
-      - run:
-          name: Run rubocop
-          command: bundle exec rubocop
-      - run:
-          name: Run brakeman
-          command: bundle exec brakeman
-      - run:
-          name: Run haml-lint
-          command: bundle exec haml-lint app/views/
-      - run:
-          name: Run scss-lint
-          command: bundle exec scss-lint app/assets/stylesheets/
   deploy:
     <<: *defaults
     steps:

README.md

@@ -108,11 +108,9 @@ Une fois `overmind` lancé, et un breakpoint `byebug` inséré dans le code, il
 
 ## Linting
 
-- Faire tourner RuboCop : `bundle exec rubocop`
+Le projet utilise plusieurs linters pour vérifier la lisibilité et la qualité code.
-- Faire tourner Brakeman : `bundle exec brakeman`
+
-- Linter les fichiers HAML : `bundle exec haml-lint app/views/`
+- Faire tourner tous les linters : `bin/rake lint`
-- Linter les fichiers SCSS : `bundle exec scss-lint app/assets/stylesheets/`
-- Linter les fichiers JavaScript : `yarn lint:js` (`yarn lint:js --fix`)
 - [AccessLint](http://accesslint.com/) tourne automatiquement sur les PRs
 
 ## Déploiement

Rakefile

@@ -5,6 +5,14 @@ require File.expand_path('config/application', __dir__)
 
 Rails.application.load_tasks
 
+task :lint do
+  sh "bundle exec rubocop"
+  sh "bundle exec haml-lint app/views/"
+  sh "bundle exec scss-lint app/assets/stylesheets/"
+  sh "bundle exec brakeman --no-pager"
+  sh "yarn lint:js"
+end
+
 task :deploy do
   domains = %w(37.187.249.111 149.202.72.152 149.202.198.6)
   domains.each do |domain|