mirror of
https://github.com/betagouv/mon-entreprise
synced 2025-02-08 21:05:01 +00:00
6de970f0e3
Outils ====== Ce commit retire le tooling de Flow, et ajoute le support de TypeScript pour les fichiers .ts et .tsx. Il n'est pas nécessaire de tout migrer d'un coup ce qui facilite la transition. On garde en effet le compilateur Babel avec un preset TypeScript (ce qui permet donc de retirer à la fois les types Flow et TypeScript) plutôt que d'utiliser le compilateur standard pour la conversion du code. Cela permet aussi de mieux s'intégrer avec d'autres outils, notamment les test-runners. Ajout d'une nouvelle commande `yarn run type-check`, intégrée dans CircleCI. Par ailleurs ajout du support de l'opérateur ?? pour donner des valeurs par défaut (nullish-coalescing-operator). Typage des libraires tierces ============================ Les principales libraires que nous utilisons ont un typage TypeScript de bon niveau, ce qui facilite l'intégration. J'ai mis à jour react-i18next et i18next afin de corriger un problème de typage. Typage du code ============== Le typage est loin d'être complet dans ce commit, en particulier il manque les types relatifs au state Redux, ainsi qu'au moteur (règle, explication). Néanmoins le typage des contextes fonctionne, en particulier sitePaths (avec un type récursif non trivial !) qui a déjà permis de détecter un lien mort. Le typage des "paths" (Components/, Règles/, etc.) fonctionne bien, y compris avec l'auto-complétion automatique des import par Typescript. TypeScript se révèle déjà bien agréable dans VSCode (auto-complétion, refacto, etc.) ! Reste à migrer progressivement le reste du code !
46 lines
898 B
YAML
46 lines
898 B
YAML
rules:
|
|
quotes:
|
|
- 1 # While https://github.com/eslint/eslint/issues/9662#issuecomment-353958854 we don't enforce this
|
|
- single
|
|
- avoidEscape: true
|
|
no-console: 1
|
|
no-restricted-globals: [2, 'length']
|
|
no-global-assign: 0
|
|
no-unsafe-negation: 0
|
|
react/prop-types: 0
|
|
react/jsx-no-target-blank: 0
|
|
react/no-unescaped-entities: 0
|
|
react/display-name: 1
|
|
react-hooks/rules-of-hooks: error
|
|
react-hooks/exhaustive-deps: warn
|
|
parser: babel-eslint
|
|
|
|
plugins:
|
|
- react
|
|
- react-hooks
|
|
env:
|
|
browser: true
|
|
commonjs: true
|
|
es6: true
|
|
settings:
|
|
react:
|
|
version: 'detect'
|
|
flowVersion: '0.92'
|
|
|
|
overrides:
|
|
- files: ['*.test.js', 'cypress/integration/**/*.js']
|
|
env:
|
|
mocha: true
|
|
|
|
globals:
|
|
process: false
|
|
|
|
extends:
|
|
- eslint:recommended
|
|
- plugin:react/recommended
|
|
- prettier
|
|
- prettier/flowtype
|
|
- prettier/react
|
|
parserOptions:
|
|
ecmaFeatures:
|
|
jsx: true
|