-
Notifications
You must be signed in to change notification settings - Fork 4
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
chore: review project conf #656
Conversation
totakoko
commented
Dec 18, 2023
•
edited
Loading
edited
- Ajout de vitest pour les tests unitaires
- Maj de la conf eslint pour supprimer un warning (on passe par next)
- Nettoyage dans le workflow CI, mise à jour des actions et pin des versions.
- Le workflow s'exécute maintenant pour tous push (de commits). Anciennement ça s'exécutait au pull request également, et c'était en doublon. Sachant que scalingo s'exécute toujours en parallèle et non à la fin si toute la CI à fonctionné.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai testé le site et rien ne semble avoir régressé. Fais l'install chez moi, idem.
Est-ce que je peux tester d'une autre façon ?
Non, c'est principalement de la conf projet. Tu peux lancer les tests avec |
Yarn test ok ! |
b158b6a
to
9156ebd
Compare
Carrément, pour l'instant j'ai juste mis un test unitaire qui sert d'exemple avec aucune dépendance donc super facile à tester. J'ai complété la doc sur les commandes importantes en dev 👉 9156ebd Dis-moi si c'est clair pour toi et si ça le sera pour quelqu'un d'autre :) |