pix icon indicating copy to clipboard operation
pix copied to clipboard

[TECH] Ajout de script magiques pour aider à la conversion d'un fichier JS en Typescript

Open Anne-Gaelle-S opened this issue 2 years ago • 1 comments

:unicorn: Problème

Renommer à la main un fichier .ts en .js et ouvrir la PR de migration de Typescript pour savoir comment s'y prendre ... c'est long et on a souvent la flemme.

:robot: Solution

Ajouter des scripts magiques pour aider à convertir un fichier .js en fichier .ts.

🌈 Remarques

Ces scripts sont là pour aider, faciliter la tâche mais ils requirent des modifications une fois le fichier transformé ! Vous devrez remplir les ? , passer un coup de linter et évidemment relire le fichier en entier pour vérifier que tout s'est bien déroulé.

/!\ Ces scripts sont un peu simplet et risquent de ne pas fonctionner sur un fichier qui a une structure "classique". Utilisez les à vos risques et périls

TODO :

  • [ ] utiliser un parser JS (ex: https://github.com/acornjs/acorn, pour tester en ligne: https://astexplorer.net/#/gist/956e939ffa7672bf370285cec8e46a8b/latest)

:100: Pour convertir vos fidèles

A partir du dossier API :

  • cd api

Lancer le script sur un usecase :

  • ./scripts/typescript/migrate-usecase-file-to-typescript.js lib/domain/usecases/<fichier-à-convertir>.js

Lancer le script sur un repository :

  • ./scripts/typescript/migrate-repository-file-to-typescript.js lib/infrastructure/repositories/<fichier-à-convertir>.js

image

Anne-Gaelle-S avatar Aug 19 '22 17:08 Anne-Gaelle-S

I'm deploying this PR to these urls:

  • App (.fr): https://app-pr4802.review.pix.fr
  • App (.org): https://app-pr4802.review.pix.org
  • Orga: https://orga-pr4802.review.pix.fr
  • Certif: https://certif-pr4802.review.pix.fr
  • Admin: https://admin-pr4802.review.pix.fr
  • API: https://api-pr4802.review.pix.fr/api/

Please check it out!

pix-service avatar Aug 19 '22 17:08 pix-service

Ouvert depuis trop longtemps

octo-topi avatar Nov 24 '22 17:11 octo-topi