facebook/TestSlide

Validate that contexts are not async defs

Offen

#143 geöffnet am 07.04.2020

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (58 Forks)auto 404
TestSlide DSLeasyenhancementgood first issuehelp wanted

Repository-Metriken

Stars
 (147 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Füge in den @context , @context.sub context und @context.shared context Dekorateuren eine Validierung hinzu, um async def Funktionen abzulehnen und einen klaren Fehler auszulösen.
Tech Stack
python
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80