Réécriture de l'entièreté des fichier JS du répertoire `/utils` en TS

Sacha338 commented 2 months ago

Checklist d'avant pull request

[x] Vous avez testé de build le projet avec vos modifications et ce build a réussi
[x] Vous respectez les conventions de codage et de nomage du projet
[x] Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
[x] Cette pull request n'est pas un duplicata d'une autre
[x] Cette pull request est prête à être revue (review) et fusionnée (merge)
[x] Cette pull request doit être fusionnée dans la branche development (le cas contraire préciser laquelle)
[x] Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
[x] Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
[x] Les détails des changements ont été décris ci-dessous
[x] Cette pull-request n'est pas une "breaking-change" (par exemple des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Explications des changements

Tous les fichiers du répertoire `/utils qui étaient en Javascript son désormais en Typescript.

Changelogs proposés

Tous les fichiers du répertoire `/utils qui étaient en Javascript son désormais en Typescript.

Vilerio commented 2 months ago

est-ce vraiment nécessaire ? ces fichiers demandent-ils un typage fort et une structure ts ?

Gabriel29306 commented 1 month ago

On fait quoi, on close ou on merge ?

Gabriel29306 commented 1 month ago

Ça va faire plus de 1 mois qu'il n'y a pas eu d'avancement.

Sacha338 commented 1 month ago

Bah je sais pas trop

Sacha338 commented 1 month ago

@Vilerio, t'en pense quoi ?

Vilerio commented 1 month ago

@Vilerio, t'en pense quoi ?

est-ce vraiment nécessaire ? ces fichiers demandent-ils un typage fort et une structure ts ?

Sacha338 commented 1 month ago

Donc je ferme ?

Vilerio commented 1 month ago

pas forcément, je demande juste et idéalement faudrait faire gaffe aux conventionalcommits à l'avenir

Sacha338 commented 1 month ago

Personnellement je trouve ça plus propre, après faudrait l'avis de quelqu'un d'autre. Je prends note pour les commits.

Vilerio commented 1 month ago

C'est pas qu'une question de propre, y a aussi une question de pertinence/compatibilité des types mais oui je suis d'accord quand même sur la propreté

Sacha338 commented 1 week ago

Je suppose qu'on merge. Ça mange pas de pain et surtout, ça ne nuit pas au code, ça l'améliore.

LeGeek01 commented 1 week ago

Vérifie que ça fonctionne avec la branch dev et si c'est bon je merge

Sacha338 commented 1 week ago

Normalement ça fonctionne, j'essaye et je te dis

Sacha338 commented 1 week ago

C’est bon, tout fonctionne

PapillonApp / Papillon