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

chore: review project conf #656

Merged
merged 5 commits into from
Dec 21, 2023
Merged

chore: review project conf #656

merged 5 commits into from
Dec 21, 2023

Conversation

totakoko
Copy link
Member

@totakoko totakoko commented Dec 18, 2023

  • 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é.

@totakoko totakoko requested a review from cspriet December 18, 2023 08:40
Copy link
Contributor

@cspriet cspriet left a 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 ?

@totakoko
Copy link
Member Author

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. D'ailleurs, je vais commencer à documenter les quelques commandes utiles pendant le développement test / lint / build.
... et résoudre le conflit du yarn.lock :)

@cspriet
Copy link
Contributor

cspriet commented Dec 21, 2023

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. D'ailleurs, je vais commencer à documenter les quelques commandes utiles pendant le développement test / lint / build. ... et résoudre le conflit du yarn.lock :)

Yarn test ok !
Je veux bien qu'à l'occaz tu m'expliques comment intégrer les tests, c'est vraiment trop bien si on peut les ajouter petit à petit. On peut se prévoir ça à la rentrée.

@totakoko totakoko force-pushed the chore/review-project-conf branch from b158b6a to 9156ebd Compare December 21, 2023 09:12
@totakoko
Copy link
Member Author

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. D'ailleurs, je vais commencer à documenter les quelques commandes utiles pendant le développement test / lint / build. ... et résoudre le conflit du yarn.lock :)

Yarn test ok ! Je veux bien qu'à l'occaz tu m'expliques comment intégrer les tests, c'est vraiment trop bien si on peut les ajouter petit à petit. On peut se prévoir ça à la rentrée.

Carrément, pour l'instant j'ai juste mis un test unitaire qui sert d'exemple avec aucune dépendance donc super facile à tester.
On pourra commencer avec quelques tests d'intégration / E2E d'API (= couche HTTP avec une vraie BDD), qui sont un bon compromis.

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 :)

@totakoko totakoko merged commit a801dca into dev Dec 21, 2023
2 checks passed
@totakoko totakoko deleted the chore/review-project-conf branch December 21, 2023 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants