PyCQA/flake8-bugbear

Proposed opinionated warning against re.match

Offen

#132 geöffnet am 13.08.2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (116 Forks)github user discovery
help wantedquestion

Repository-Metriken

Stars
 (1.116 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4h 17m) (3 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandenen Warnimplementierungen in flake8 bugbear, um das Muster zu verstehen, und erstellen Sie dann eine neue Warnung, die die Verwendung von re.match kennzeichnet, wenn möglicherweise re.search beabsichtigt war. Berücksichtigen Sie Fälle wie die Überprüfung, ob das Regex Muster für den Abgleich am Anfang einer Zeichenfolge verwendet wird.
Tech Stack
python
Domain
devtools
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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.
70