PyCQA/flake8-bugbear

Proposed opinionated warning against re.match

Ouverte

#132 ouverte le 13 août 2020

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)Python (116 forks)github user discovery
help wantedquestion

Métriques du dépôt

Stars
 (1 116 étoiles)
Métriques de merge PR
 (Merge moyen 4h 17m) (3 PRs mergées en 30 j)

Description

Technically, using re.match is valid. Realistically, the author probably meant .search and is going to lose a ton of time trying to figure out why her regex isn't working.

I'm speaking purely in compassion toward more junior developers; I would obviously never make such an elementary mistake myself. https://twitter.com/catherinedevlin/status/1257507010041520139

If you think you're likely to accept it, I'd be interested in authoring such a warning.

Guide contributeur

Direction de recherche
Examinez les implémentations d'avertissement existantes dans flake8 bugbear pour comprendre le modèle, puis créez un nouvel avertissement qui signale l'utilisation de re.match là où re.search était peut être prévu. Envisagez des cas comme vérifier si le motif regex est utilisé pour la correspondance au début d'une chaîne.
Stack technique
python
Domaine
devtools
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70