cuentoneta icon indicating copy to clipboard operation
cuentoneta copied to clipboard

Eliminar archivo .eslintrc.json

Open erikgiovani opened this issue 1 year ago • 4 comments

Descripción

Esperar el soporte total de eslint 9 de los plugins para borrar el archivo .eslintrc.json y quedarnos solo con el eslint.config.js y remover el uso de FlatCompat para tener un archivo de configuración mas limpio.

Plugins a actualizar

  • [ ] eslint-plugin-storybook
  • [ ] eslint-plugin-testing-library
  • [ ] eslint-plugin-jest-dom

Recursos

Estado de los plugins a migrar

Como referencia, en el repo de eslint están siguiendo los plugins y su soporte, Tracking: ESLint v9 support.

erikgiovani avatar Sep 18 '24 14:09 erikgiovani

¿Podremos tener la lista de todos los plugins a migrar? De esa manera podemos ir haciendo una migración ordenada.

rolivencia avatar Sep 18 '24 16:09 rolivencia

Acá la tienes Rami.

Como referencia, en el repo de eslint están siguiendo los plugins y su soporte, Tracking: ESLint v9 support.

Addin avatar Sep 18 '24 16:09 Addin

Acá la tienes Rami.

Como referencia, en el repo de eslint están siguiendo los plugins y su soporte, Tracking: ESLint v9 support.

Buena referencia 👌

erikgiovani avatar Sep 18 '24 16:09 erikgiovani

¿Podremos tener la lista de todos los plugins a migrar? De esa manera podemos ir haciendo una migración ordenada.

Ready 👍

erikgiovani avatar Sep 18 '24 16:09 erikgiovani

Parece que ya tienen lista la PR para proceder con la actualización faltante en el repo de storybookjs/eslint-plugin-storybook: https://github.com/storybookjs/eslint-plugin-storybook/pull/156

cc @ErikGIovani @Addin

rolivencia avatar Oct 08 '24 15:10 rolivencia

Parece que ya tienen lista la PR para proceder con la actualización faltante en el repo de storybookjs/eslint-plugin-storybook: storybookjs/eslint-plugin-storybook#156

cc @ErikGIovani @Addin

Excelente 🥳, solo falta https://github.com/testing-library/eslint-plugin-testing-library/pull/928

erikgiovani avatar Oct 12 '24 22:10 erikgiovani

@rolivencia hace tres días hicieron merge del pull request de eslint-plugin-testing-library para el soporte de eslint 9, espero la actualización de nx y angular y empiezo con el issue en caso de que existan cambios con alguna de sus dependencias 👍

erikgiovani avatar Oct 20 '24 19:10 erikgiovani

Listo el trabajo de #1035 para la actualización de Angular @erikgiovani. Tuve que proceder con la actualización a ESLint 9, y la de los plugins, debido a que sino me era imposible actualizar Nx a la última versión.

La dependencia eslint-plugin-testing-library está dando un warning por incompatibilidad de versiones. Tenlo en cuenta a la hora de abordar este issue, dado que quizá el soporte está cerca. En todo caso, si lo crees conveniente, podemos dejar un issue abierto específicamente para abordar este tema.

rolivencia avatar Oct 23 '24 22:10 rolivencia

@rolivencia gracias por la recomendación, lo tendré en cuenta 🥳

erikgiovani avatar Oct 24 '24 02:10 erikgiovani