hound icon indicating copy to clipboard operation
hound copied to clipboard

Python: Conflicts vs. the new flake8

Open cclauss opened this issue 7 years ago • 3 comments

See: https://github.com/home-assistant/home-assistant/pulls/cclauss

flake8==3.7.5 now check for undefined names (F821) in the type hints in comments. Hound is using flake8==3.6.0 so it disagrees with the current flake8 and raises 'typing.List' imported but unused errors on fixes.

cclauss avatar Feb 01 '19 10:02 cclauss

@cclauss Any idea if 3.7.4 is backwards compatible? Would there any breakage if we upgrade the default to 3.7.4 vs providing 3.7.4 as an opt-in upgrade?

gylaz avatar Feb 01 '19 17:02 gylaz

https://github.com/PyCQA/flake8/tree/master/docs/source/release-notes 3.7.0 upgraded pyfakes and pycodestyle so it is unclear to me.

Opt-in upgrade sounds like the best bet. Perhaps a version number in .hound.yml

flake8:
  enabled: true
  flake8_version: 3.7.4

cclauss avatar Feb 01 '19 18:02 cclauss

flake8_version: 3.7.4 --> 3.7.5 http://flake8.pycqa.org/en/latest/release-notes/index.html

cclauss avatar Feb 04 '19 18:02 cclauss