Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Mise à jour de ESLint vers la V9 (PIX-15823) #10870

Merged
merged 7 commits into from
Dec 30, 2024

Conversation

mcampourcy
Copy link
Contributor

@mcampourcy mcampourcy commented Dec 20, 2024

⚠️ Le plugin json retourne plein d'erreurs sur les fichiers de trad qui ont des balises
⚠️ Le build casse à cause d'ESM

🎄 Problème

ESLint a publié une version 9 de la lib, version qui implique pas mal de changements cassants. Il faudrait mettre à jour pour éviter de se traîner trop de dette technique.

🎁 Proposition

  • Montée de version de ESLint en v9
  • Modification de la config en flatConfig

🧦 Remarques

🎅 Pour tester

Vérifier que les tests passent

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@mcampourcy mcampourcy force-pushed the pix-15823-upgrade-eslint-to-v9 branch 2 times, most recently from 54dc54d to ee23963 Compare December 20, 2024 08:26
@yannbertrand
Copy link
Member

Il reste une erreur avec le ember-cli-build.js qui est en CJS. Je ne sais pas si c'est possible de le passer en ESM dès maintenant ? Peut être que c'est trop tôt pour mettre le "type": "module" 🤔

@mcampourcy mcampourcy force-pushed the pix-15823-upgrade-eslint-to-v9 branch 4 times, most recently from f889561 to e97574e Compare December 20, 2024 15:11
@xav-car
Copy link
Contributor

xav-car commented Dec 23, 2024

Il reste une erreur avec le ember-cli-build.js qui est en CJS. Je ne sais pas si c'est possible de le passer en ESM dès maintenant ? Peut être que c'est trop tôt pour mettre le "type": "module" 🤔

Effectivement, cela ne semble pas possible actuellement, de ce que j'ai pu tenté sur PixOrga. en passant les fichiers esm en mjs. tout semble ok côté Orga

@xav-car xav-car force-pushed the pix-15823-upgrade-eslint-to-v9 branch 2 times, most recently from d24467e to 26eb188 Compare December 23, 2024 14:06
@xav-car xav-car marked this pull request as ready for review December 23, 2024 14:32
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15823-upgrade-eslint-to-v9 branch from 26eb188 to c49701a Compare December 30, 2024 09:35
@pix-service-auto-merge pix-service-auto-merge merged commit b6468d6 into dev Dec 30, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15823-upgrade-eslint-to-v9 branch December 30, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants