OpenZeppelin/openzeppelin-contracts

Consider adding an ESLint rule for no async in describe blocks

Chiusa

#4943 aperta il 6 mar 2024

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (11.560 fork)batch import
good first issuetests

Metriche repository

Star
 (23.941 stelle)
Metriche merge PR
 (Merge medio 42g 15h) (16 PR mergiate in 30 g)

Descrizione

From @fvictorio:

I forgot to mention:

  • The wrong async keywords in describe functions can be prevented with this rule.
  • The missing await keywords are harder. Ideally you'd use typescript and the no-floating-promises rule, but I understand that that would be a huge undertaking.

Originally posted by @fvictorio in https://github.com/OpenZeppelin/openzeppelin-contracts/issues/4942#issuecomment-1980874051

We're considering adding only the ESLint rule and not migrating to Typescript to catch floating promises. However, we're open to alternatives.

Guida contributor

Direzione di ricerca
Installa eslint plugin mocha e aggiungi la regola 'no async describe' alla configurazione ESLint.
Tech stack
javascript
Dominio
developer experiencetooling
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
javascript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90