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] Feat add back translations lint #10369

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Oct 19, 2024

🦄 Problème

Le linting des traductions a été supprimé dans api et mon-pix avec 2 PR et car nous n'arrivions pas à utiliser le plugin ESLint avec le format flat config d'ESLint :

Mais le linting des traductions est très pratique, car notamment il ordonne par tri alphabétique les traductions, ce qui diminue les conflits potentiels sur les PR.

🤖 Proposition

Remettre en place les scripts npm lint:translations et lint:translations:fix de manière à ce qu'on les trouve dans tous les projets ayant des traductions.

La remise en place des scripts npm lint:translations et lint:translations:fix dans api et mon-pix utilise toujours https://github.com/godaddy/eslint-plugin-i18n-json mais en se basant sur la solution fournie dans godaddy/eslint-plugin-i18n-json#62 puisque eslint-plugin-i18n-json ne supporte pas de base flat config.

🌈 Remarques

On pourra simplifier la configuration relative à eslint-plugin-i18n si/quand ce plugin sera de base compatible avec flat config.

💯 Pour tester

  1. Vérifier que la CI passe
  2. Vérifier que la commande npm run lint:translations:fix est bien fonctionnelle :
    2.1 Modifier par exemple le fichier translations/fr.json en inversant l'ordre de certaines traductions
    2.2 Exécuter la commande npm run lint:translations:fix
    2.3 Constater que le linter a remis les traductions dans l'ordre

@lego-technix lego-technix self-assigned this Oct 19, 2024
@lego-technix lego-technix marked this pull request as ready for review October 19, 2024 19:31
@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 :

@lego-technix lego-technix force-pushed the feat-add-back-translations-lint branch from 95463bb to 3a968de Compare October 20, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 👀 Tech Review Needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants