bamlab / react-native-project-config

Monorepo with packages for setting up ESLint, Typescript, Prettier and Jest.
8 stars 0 forks source link

chore: delete custom rule to await user event #94

Open pierrezimmermannbam opened 1 year ago

pierrezimmermannbam commented 1 year ago

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

AntoineThibi commented 11 months ago

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 commented 11 months ago

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 commented 11 months ago

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

pierrezimmermannbam commented 11 months ago

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