flake8-bugbear icon indicating copy to clipboard operation
flake8-bugbear copied to clipboard

Proposed opinionated warning against re.match

Open catherinedevlin opened this issue 5 years ago • 1 comments

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.

catherinedevlin avatar Aug 13 '20 16:08 catherinedevlin

I'd like another maintainer to comment, but if this was disabled by default I'd accept the PR as we've all done this.

cooperlees avatar Aug 13 '20 18:08 cooperlees