MichaelKim0407 / flake8-use-fstring

MIT License
51 stars 7 forks source link

Add check for missing `f` prefix for f-strings. #2

Closed plinss closed 4 years ago

plinss commented 4 years ago

Fixes #1

plinss commented 4 years ago

Ok, I think I addressed all your concerns. Take your time deciding whether or not to include this, I mostly wrote it for my own safety checks, but felt it would be worth contributing back.

I actually debated if this should be a completely separate plugin, but I felt that functionality fit with responding to the errors generated here.

If nothing else, I'm planning on make a few flake8 extensions myself, so this was a good learning experience.