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

Merged
merged 5 commits into from
Oct 21, 2024

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 : les PR #8995 et #9847

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
    2.4 Faire les mêmes opération avec un fichier généré par phrase, par exemple : translations/fr.json. Vérifier que les commandes npm run lint:translations:fix et npm run lint:translations:fix ignorent la modification.

@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
@aurelie-crouillebois aurelie-crouillebois added the Func Review OK PO validated functionally the PR label Oct 21, 2024
Copy link
Member

@yannbertrand yannbertrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, un contournement ! Merci :)

@pix-service-auto-merge pix-service-auto-merge merged commit 6c394bc into dev Oct 21, 2024
20 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the feat-add-back-translations-lint branch October 21, 2024 07:35
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 Func Review OK PO validated functionally the PR 🚀 Ready to Merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants