PyCQA/flake8-bugbear

Proposed opinionated warning against re.match

Aperta

#132 aperta il 13 ago 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (116 fork)github user discovery
help wantedquestion

Metriche repository

Star
 (1116 stelle)
Metriche merge PR
 (Merge medio 4h 17m) (3 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina le implementazioni di avviso esistenti in flake8 bugbear per comprendere il pattern, quindi crea un nuovo avviso che segnali l'uso di re.match quando potrebbe essere inteso re.search. Considera casi come il controllo se il pattern regex viene utilizzato per la corrispondenza all'inizio di una stringa.
Tech stack
python
Dominio
devtools
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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.
70