Improved pre-commit and linters config

modoboa · Feb 21, 2024 · f1d89bd · f1d89bd
1 parent 692871b
commit f1d89bd
Show file tree

Hide file tree

Showing 141 changed files with 10,434 additions and 8,089 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -9,20 +9,19 @@ repos:
         language: system
         types: [python]
         args: ["-rn", "-sn", "--fail-under=5"]
+      - id: prettier
+        name: prettier
+        entry: prettier --config frontend/.prettierrc.json --write --ignore-unknown
+        language: node
+        types: [javascript]
+        additional_dependencies: ["prettier"]
+      - id: eslint
+        name: eslint
+        language: node
+        types: [javascript]
+        entry: eslint --fix 
+        additional_dependencies: ["eslint-plugin-vue"]
   - repo: https://github.com/psf/black
     rev: "24.2.0"
     hooks:
       - id: black
-  - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: "v9.0.0-beta.0"
-    hooks:
-      - id: eslint
-        files: \.(js|vue)$
-        types: [file]
-        additional_dependencies:
-          - eslint
-          - eslint-plugin-vue
-  - repo: https://github.com/pre-commit/mirrors-prettier
-    rev: "v4.0.0-alpha.8" # Use the sha / tag you want to point at
-    hooks:
-      - id: prettier
diff --git a/frontend/.eslintrc.js b/frontend/.eslintrc.js
@@ -1,17 +1,17 @@
 module.exports = {
-  root: true,
-  env: {
-    node: true,
-  },
-  extends: ['plugin:vue/vue3-recommended', 'eslint:recommended', 'prettier'],
-  rules: {
-    'vue/component-name-in-template-casing': [
-      'error',
-      'PascalCase',
-      {
-        registeredComponentsOnly: true,
-        ignores: [],
-      },
-    ],
-  },
+    root: true,
+    env: {
+        node: true,
+    },
+    extends: ['plugin:vue/vue3-recommended', 'eslint:recommended'],
+    rules: {
+        'vue/component-name-in-template-casing': [
+            'error',
+            'PascalCase',
+            {
+                registeredComponentsOnly: true,
+                ignores: [],
+            },
+        ],
+    },
 }
diff --git a/.prettierrc.json → frontend/.prettierrc.json b/.prettierrc.json → frontend/.prettierrc.json
diff --git a/frontend/gettext.config.js b/frontend/gettext.config.js
@@ -1,24 +1,24 @@
 module.exports = {
-  output: {
-    path: './src/locale',
-    locales: [
-      'en',
-      'br',
-      'cs',
-      'de',
-      'el',
-      'es',
-      'fi',
-      'fr',
-      'it',
-      'ja',
-      'nl',
-      'pl',
-      'pt',
-      'ru',
-      'sv',
-      'tr',
-      'zh',
-    ],
-  },
+    output: {
+        path: './src/locale',
+        locales: [
+            'en',
+            'br',
+            'cs',
+            'de',
+            'el',
+            'es',
+            'fi',
+            'fr',
+            'it',
+            'ja',
+            'nl',
+            'pl',
+            'pt',
+            'ru',
+            'sv',
+            'tr',
+            'zh',
+        ],
+    },
 }
diff --git a/frontend/package.json b/frontend/package.json
@@ -33,7 +33,6 @@
   "devDependencies": {
     "@vitejs/plugin-vue": "^5.0.4",
     "eslint": "^8.56.0",
-    "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-vue": "^9.21.1",
     "prettier": "^3.1.1",
     "sass": "^1.69.5",

diff --git a/frontend/src/App.vue b/frontend/src/App.vue
@@ -1,5 +1,5 @@
 <template>
-  <router-view />
+    <router-view />
 </template>
 
 <script setup></script>
diff --git a/frontend/src/api/account.js b/frontend/src/api/account.js
@@ -3,45 +3,45 @@ import repository from './repository'
 const resource = 'account'
 
 export default {
-  getMe() {
-    return repository.get(`/${resource}/me/`)
-  },
-  checkPassword(value) {
-    return repository.post(`/${resource}/me/password/`, { password: value })
-  },
-  verifyTFACode(code) {
-    const payload = { code }
-    return repository.post(`${resource}/tfa/verify/`, payload)
-  },
-  startTFASetup() {
-    return repository.post(`${resource}/tfa/setup/`)
-  },
-  getKeyForTFASetup() {
-    return repository.get(`${resource}/tfa/setup/key/`)
-  },
-  finalizeTFASetup(pinCode) {
-    const payload = { pin_code: pinCode }
-    return repository.post(`${resource}/tfa/setup/check/`, payload)
-  },
-  disableTFA(data) {
-    return repository.post(`${resource}/tfa/disable/`, data)
-  },
-  resetRecoveryCodes(data) {
-    return repository.post(`${resource}/tfa/reset_codes/`, data)
-  },
-  getForward() {
-    return repository.get(`${resource}/forward/`)
-  },
-  setForward(data) {
-    return repository.post(`${resource}/forward/`, data)
-  },
-  getAPIToken() {
-    return repository.get(`${resource}/api_token/`)
-  },
-  createAPIToken() {
-    return repository.post(`${resource}/api_token/`)
-  },
-  deleteAPIToken() {
-    return repository.delete(`${resource}/api_token/`)
-  },
+    getMe() {
+        return repository.get(`/${resource}/me/`)
+    },
+    checkPassword(value) {
+        return repository.post(`/${resource}/me/password/`, { password: value })
+    },
+    verifyTFACode(code) {
+        const payload = { code }
+        return repository.post(`${resource}/tfa/verify/`, payload)
+    },
+    startTFASetup() {
+        return repository.post(`${resource}/tfa/setup/`)
+    },
+    getKeyForTFASetup() {
+        return repository.get(`${resource}/tfa/setup/key/`)
+    },
+    finalizeTFASetup(pinCode) {
+        const payload = { pin_code: pinCode }
+        return repository.post(`${resource}/tfa/setup/check/`, payload)
+    },
+    disableTFA(data) {
+        return repository.post(`${resource}/tfa/disable/`, data)
+    },
+    resetRecoveryCodes(data) {
+        return repository.post(`${resource}/tfa/reset_codes/`, data)
+    },
+    getForward() {
+        return repository.get(`${resource}/forward/`)
+    },
+    setForward(data) {
+        return repository.post(`${resource}/forward/`, data)
+    },
+    getAPIToken() {
+        return repository.get(`${resource}/api_token/`)
+    },
+    createAPIToken() {
+        return repository.post(`${resource}/api_token/`)
+    },
+    deleteAPIToken() {
+        return repository.delete(`${resource}/api_token/`)
+    },
 }
diff --git a/frontend/src/api/accounts.js b/frontend/src/api/accounts.js
@@ -3,25 +3,25 @@ import repository from './repository'
 const resource = 'accounts'
 
 export default {
-  getAll({ domain, role }) {
-    return repository.get(`/${resource}/`, { params: { domain, role } })
-  },
-  get(accountId) {
-    return repository.get(`${resource}/${accountId}/`)
-  },
-  getRandomPassword() {
-    return repository.get(`${resource}/random_password/`)
-  },
-  validate(data) {
-    return repository.post(`${resource}/validate/`, data)
-  },
-  create(data) {
-    return repository.post(`${resource}/`, data)
-  },
-  patch(accountId, data) {
-    return repository.patch(`${resource}/${accountId}/`, data)
-  },
-  delete(accountId, data) {
-    return repository.post(`${resource}/${accountId}/delete/`, data)
-  },
+    getAll({ domain, role }) {
+        return repository.get(`/${resource}/`, { params: { domain, role } })
+    },
+    get(accountId) {
+        return repository.get(`${resource}/${accountId}/`)
+    },
+    getRandomPassword() {
+        return repository.get(`${resource}/random_password/`)
+    },
+    validate(data) {
+        return repository.post(`${resource}/validate/`, data)
+    },
+    create(data) {
+        return repository.post(`${resource}/`, data)
+    },
+    patch(accountId, data) {
+        return repository.patch(`${resource}/${accountId}/`, data)
+    },
+    delete(accountId, data) {
+        return repository.post(`${resource}/${accountId}/delete/`, data)
+    },
 }
diff --git a/frontend/src/api/admin.js b/frontend/src/api/admin.js
@@ -1,7 +1,7 @@
 import repository from './repository'
 
 export default {
-  getComponentsInformation() {
-    return repository.get('/admin/components/')
-  },
+    getComponentsInformation() {
+        return repository.get('/admin/components/')
+    },
 }
diff --git a/frontend/src/api/alarms.js b/frontend/src/api/alarms.js
@@ -3,18 +3,18 @@ import repository from './repository'
 const resource = 'alarms'
 
 export default {
-  getAll(params) {
-    return repository.get(`${resource}/`, { params })
-  },
-  switchStatus(alarmId, data) {
-    return repository.patch(`${resource}/${alarmId}/switch/`, data)
-  },
-  delete(alarmId) {
-    return repository.delete(`${resource}/${alarmId}/`)
-  },
-  bulkDelete(alarmsIds) {
-    return repository.delete(`${resource}/bulk_delete/`, {
-      params: { ids: alarmsIds },
-    })
-  },
+    getAll(params) {
+        return repository.get(`${resource}/`, { params })
+    },
+    switchStatus(alarmId, data) {
+        return repository.patch(`${resource}/${alarmId}/switch/`, data)
+    },
+    delete(alarmId) {
+        return repository.delete(`${resource}/${alarmId}/`)
+    },
+    bulkDelete(alarmsIds) {
+        return repository.delete(`${resource}/bulk_delete/`, {
+            params: { ids: alarmsIds },
+        })
+    },
 }
diff --git a/frontend/src/api/aliases.js b/frontend/src/api/aliases.js
@@ -3,25 +3,25 @@ import repository from './repository'
 const resource = 'aliases'
 
 export default {
-  getAll() {
-    return repository.get(`${resource}/`)
-  },
-  get(aliasId) {
-    return repository.get(`${resource}/${aliasId}/`)
-  },
-  getRandomAddress() {
-    return repository.get(`${resource}/random_address/`)
-  },
-  validate(data) {
-    return repository.post(`${resource}/validate/`, data)
-  },
-  create(data) {
-    return repository.post(`${resource}/`, data)
-  },
-  patch(aliasId, data) {
-    return repository.patch(`${resource}/${aliasId}/`, data)
-  },
-  delete(aliasId) {
-    return repository.delete(`${resource}/${aliasId}/`)
-  },
+    getAll() {
+        return repository.get(`${resource}/`)
+    },
+    get(aliasId) {
+        return repository.get(`${resource}/${aliasId}/`)
+    },
+    getRandomAddress() {
+        return repository.get(`${resource}/random_address/`)
+    },
+    validate(data) {
+        return repository.post(`${resource}/validate/`, data)
+    },
+    create(data) {
+        return repository.post(`${resource}/`, data)
+    },
+    patch(aliasId, data) {
+        return repository.patch(`${resource}/${aliasId}/`, data)
+    },
+    delete(aliasId) {
+        return repository.delete(`${resource}/${aliasId}/`)
+    },
 }
diff --git a/frontend/src/api/auth.js b/frontend/src/api/auth.js
@@ -1,16 +1,16 @@
 import repository from './repository'
 
 export default {
-  requestToken(data) {
-    return repository.post('/token/', data)
-  },
-  recoverPassword(data) {
-    return repository.post('/password_reset/', data)
-  },
-  checkSmsTotp(data) {
-    return repository.post('/sms_totp/', data)
-  },
-  changePassword(data) {
-    return repository.post('/reset_confirm/', data)
-  },
+    requestToken(data) {
+        return repository.post('/token/', data)
+    },
+    recoverPassword(data) {
+        return repository.post('/password_reset/', data)
+    },
+    checkSmsTotp(data) {
+        return repository.post('/sms_totp/', data)
+    },
+    changePassword(data) {
+        return repository.post('/reset_confirm/', data)
+    },
 }