facebook/TestSlide

Simplify typehints

Open

#248 ouverte le 2 oct. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (58 forks)auto 404
easyenhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (147 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

As per @fornellas comments on #246 do these:

  • Unions of BaseException & other exception types (eg: Exception). Shouldn't BaseException be enough? All Python exception are subclasses of it, I think it should work just fine with only it.

  • There are lots of Callable types (no args, no return) or Callable[[a,b,c...], d], mostly related to DSL hooks, such as examples, before / after hooks etc. TBH, this PR is awesome as it is, but I was thinking about having custom types such as ExampleFunction = Callable[[_ContextData], None] (non-blocking here to be clear).

Guide contributeur

Direction de recherche
Consultez le contexte de la PR #246. Examinez les indications de type actuelles dans la gestion des exceptions et les signatures callable. Simplifiez les unions en BaseException et définissez des types Callable personnalisés comme suggéré.
Stack technique
python
Domaine
testing
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70