eslint: grunt integration

manage

@@ -665,8 +665,6 @@ themes.oscar() {
 themes.simple() {
     build_msg GRUNT "theme: simple"
     npm --prefix searx/static/themes/simple run build
-    # just report eslint issues but do not break the build (--force)
-    npm --prefix searx/static/themes/simple run eslint --force
     dump_return $?
 }
 

searx/static/themes/simple/.eslintrc.json

@@ -5,8 +5,7 @@
     },
     "extends": "eslint:recommended",
     "parserOptions": {
-        "ecmaVersion": 12,
-        "sourceType": "module"
+        "ecmaVersion": 12
     },
     "rules": {
     }

searx/static/themes/simple/gruntfile.js

@@ -9,15 +9,13 @@ module.exports = function(grunt) {
     watch: {
       scripts: {
         files: ['src/**'],
-        tasks: ['jshint', 'copy', 'concat', 'uglify', 'less:development', 'less:production']
+        tasks: ['eslint', 'copy', 'concat', 'uglify', 'less:development', 'less:production']
       }
     },
-    jshint: {
-      files: ['src/js/main/*.js', 'src/js/head/*.js', '../__common__/js/*.js'],
-    },
     eslint: {
       options: {
-        configFile: '.eslintrc.json'
+        configFile: '.eslintrc.json',
+        failOnError: false
       },
       target: [
         'src/js/main/*.js',
@@ -204,8 +202,7 @@ module.exports = function(grunt) {
   grunt.registerTask('test', ['jshint']);
 
   grunt.registerTask('default', [
-    // 'eslint',
-    'jshint',
+    'eslint',
     'stylelint',
     'copy',
     'concat',

searx/static/themes/simple/package.json

@@ -1,6 +1,5 @@
 {
   "devDependencies": {
-    "eslint": "^7.32.0",
     "grunt-cli": "^1.4.3",
     "grunt": "~1.4.1",
     "grunt-contrib-copy": "^1.0.0",
@@ -14,7 +13,7 @@
     "grunt-eslint": "^23.0.0",
     "grunt-webfont": "^1.7.2",
     "ionicons-npm": "^2.0.1",
-    "jslint": "^0.12.1",
+    "eslint": "^7.32.0",
     "less": "^4.1.1",
     "less-plugin-clean-css": "^1.5.1",
     "stylelint": "^13.13.1",