omz / Pythonista-Issues

Issue tracker for Pythonista (iOS)
169 stars 14 forks source link

Merge Analyze & Check Style #357

Open zrzka opened 7 years ago

zrzka commented 7 years ago

This issue renders #350, #349, #352 as obsolete and #351 (editor.analyze) can do both things.

I just released Analyze & Check Style in Black Mamba. What it does? One shortcut for:

We can end up with more than one annotation on a line. That's okay, I'm displaying it in this way:

Also it should:

Check attached screenshot:

img_0091

cclauss commented 7 years ago

http://flake8.pycqa.org is a wrapper around both pyflakes and pep8 for other platforms.

https://github.com/openstack-dev/hacking is a wrapper around flake8 adding even more error checks.

zrzka commented 7 years ago

I checked Pythonista frameworks and I didn't find flake8. So I decided to go with pyflakes & pep8, which are available (not forcing users to install dependencies). But I agree it would be nice if we can have #85 fixed and flake8 will be used for editor.analyze / Wrench & Analyze.

cclauss commented 7 years ago

Correct. Flake8 is not currently part of Pythonista so your choice of pyflakes & pep8 was the right way to go.