search

civisanalytics / civis-python

Civis API Python Client
BSD 3-Clause "New" or "Revised" License
34 stars 26 forks source link

fix some minor issues that flake8 3.8 finds #394

Closed mheilman closed 4 years ago

mheilman commented 4 years ago

related to #393

mheilman commented 4 years ago

Ha, yeah, I was wondering the same thing. It means visually ambiguous (e.g., "l" vs "I"), so it kinda makes sense (though fixed width fonts usually render those differently: l vs I).

https://www.flake8rules.com/rules/E741.html

stephen-hoover commented 4 years ago

Ha, yeah, I was wondering the same thing. It means visually ambiguous (e.g., "l" vs "I"), so it kinda makes sense (though fixed width fonts usually render those differently: l vs I).

https://www.flake8rules.com/rules/E741.html

That makes a lot of sense, now that you point it out. I'm happy to have that check in.