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 Ajouter la règle de lint mocha/no-exclusive-tests (PIX-4085). #3887

Merged

Conversation

ajoanny
Copy link

@ajoanny ajoanny commented Dec 29, 2021

🎄 Problème

Les tests avec un .only ne cassent pas la CI
Régression suite à #3651

🎁 Solution

Rajouter la règle de lint : mocha/no-exclusive-tests

🌟 Remarques

Des infos supplémentaires, trucs et astuces ?

🎅 Pour tester

Mettre un only sur un test et lancer le linter

@ajoanny ajoanny self-assigned this Dec 29, 2021
@ajoanny ajoanny added 👀 Tech Review Needed cross-team Toutes les équipes de dev team-acces team-captains This is your captain speaking team-certif team-contenu PR relatives à l'équipe Contenu team-evaluation PR relatives à l'expérience d'évaluation team-prescription labels Dec 29, 2021
@ajoanny ajoanny force-pushed the pix-4085-add-eslint-rules-no-exclusive-test branch from 3ed61fc to 656fb08 Compare December 29, 2021 15:57
Copy link
Contributor

@mickaelalibert mickaelalibert left a comment

Choose a reason for hiding this comment

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

👍

@pix-service
Copy link
Contributor

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 🚀 Ready to Merge team-acces team-captains This is your captain speaking team-certif team-contenu PR relatives à l'équipe Contenu team-evaluation PR relatives à l'expérience d'évaluation team-prescription Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants