facebook/TestSlide

Validate that contexts are not async defs

Aperta

#143 aperta il 7 apr 2020

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Python (58 fork)auto 404
TestSlide DSLeasyenhancementgood first issuehelp wanted

Metriche repository

Star
 (147 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

This should yield a failure:

@context
def top(context):
  @context.sub_context
  async def sub(context):
     ...

But is currently being silently ignored. Let's turn this into a failure.

Check:

  • @context
  • @context.sub_context
  • @context.shared_context

Guida contributor

Direzione di ricerca
Aggiungi una validazione nei decoratori @context, @context.sub context e @context.shared context per rifiutare le funzioni async def sollevando un errore chiaro.
Tech stack
python
Dominio
testing
Tipo issue
Bug
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80