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: delete custom rule to await user event #94

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pierrezimmermannbam
Copy link
Contributor

This rule is now obsolete because the @typescript-eslint/no-floating-promises rule works for that use case

@github-actions github-actions bot added no examples 📝 📏 eslint-plugin Issue concerning the eslint plugin labels Nov 10, 2023
@pierrezimmermannbam pierrezimmermannbam force-pushed the chore/deleteRuleToAwaitUserEvent branch from 7cb7b58 to 9b70fe9 Compare November 10, 2023 16:52
@github-actions github-actions bot removed no examples 📝 📏 eslint-plugin Issue concerning the eslint plugin labels Nov 10, 2023
@AntoineThibi
Copy link
Contributor

On peut merger cette PR ?

PS : la règle @bam.tech/await-user-event a une issue (et je ne sais pas où elle est déclaré et où on peut faire l'issue dessus (et s'il y en a vraiment besoin))

// setup est une méthode synchrone, pourtant on a l'erreur : userEvent calls should be preceded by 'await'
const user = userEvent.setup()

@pierrezimmermannbam
Copy link
Contributor Author

Après réflexion je sais pas si on veut la supprimer, je la retirerais du recommended mais ça peut être commode pour des projets sur lesquels l'adoption de no-floating-promises est un peu plus compliquée

@MattAgn
Copy link

MattAgn commented Dec 15, 2023

@pierrezimmermannbam on s'était pas dit que sinon les projets pouvaient appliquer la règle no-floating-promise que sur les tests ?

@pierrezimmermannbam
Copy link
Contributor Author

Oui c'est une possibilité en effet, le seul désavantage c'est que ça nécessite un setup eslint un peu plus complexe

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.

3 participants