bamlab/react-native-project-config

[RFC]: remove import/namespace rule

pierrezimmermannbam opened this issue · 3 comments

Why?

Si je check le temps d'exécution du lint sur mon projet j'ai le résultat suivant:

Capture d’écran 2023-09-22 à 14 30 18

Cette règle prend un tiers du temps de lint, plus que prettier !

Surtout, elle me parait inutile si on utilise typescript. Si j'essaye de faire ce que la règle interdit, j'ai une erreur ts

Capture d’écran 2023-09-22 à 14 29 21

Si elle est couteuse est peu utile, on peut la désactiver

Rule Documentation 📜

https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/namespace.md

Config Selection 🛠

recommended

(Optional) Additional Details 📝

No response

Y a des fortes chances que la mesure du temps soit biaisée parce que c'est la 1ère règle de eslint-plugin-import qui est appliquée... T'as mesuré la diff de temps quand tu la retire ?

A tester : si on enlève la règle, est ce que le temps n'est quand même pas le même ? hypothèse : la première règle est plus longue à cause de la mise en route @pierrezimmermannbam

=> on enlève quand même parce qu'elle fait doublon avec TS ? @pierrezimmermannbam creuse

Il semblerait que ça ne prenne quasiment pas de temps finalement, autant garder la règle pour ne pas alourdir la config dans ce cas