Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No support for flake8 3.0 #2

Closed
The-Compiler opened this issue Jun 26, 2016 · 9 comments
Closed

No support for flake8 3.0 #2

The-Compiler opened this issue Jun 26, 2016 · 9 comments

Comments

@The-Compiler
Copy link

The-Compiler commented Jun 26, 2016

I just tried using this with flake8 3.0.0b1 and got:

Traceback (most recent call last):
  File "/usr/lib64/python3.5/runpy.py", line 170, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib64/python3.5/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/__main__.py", line 4, in <module>
    cli.main()
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/main/cli.py", line 16, in main
    app.run(argv)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/main/application.py", line 292, in run
    self._run(argv)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/main/application.py", line 278, in _run
    self.initialize(argv)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/main/application.py", line 269, in initialize
    self.register_plugin_options()
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/main/application.py", line 149, in register_plugin_options
    self.check_plugins.register_options(self.option_manager)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/plugins/manager.py", line 378, in register_options
    list(self.manager.map(call_register_options))
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/plugins/manager.py", line 252, in map
    yield func(self.plugins[name], *args, **kwargs)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/plugins/manager.py", line 348, in generated_function
    return method(optmanager, *args, **kwargs)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/flake8/plugins/manager.py", line 198, in register_options
    add_options(optmanager)
  File "/home/florian/proj/qutebrowser/git/.tox/flake8/lib/python3.5/site-packages/ebb_lint/flake8.py", line 195, in add_options
    parser.config_options.append('hard-max-line-length')
AttributeError: 'OptionManager' object has no attribute 'config_options'

flake8 documentation about the change: http://flake8.pycqa.org/en/latest/plugin-development/cross-compatibility.html#option-handling-on-flake8-2-and-3

Related fixes in pep8-naming:

PyCQA/pep8-naming@b4c3ebd
PyCQA/pep8-naming@dd623fc

@sigmavirus24
Copy link

@habnabit
Copy link
Collaborator

Ugh.

I guess I'll deal with this soon.

@sigmavirus24
Copy link

@habnabit I'm hoping to be able to help out with this and send a PR.

@The-Compiler
Copy link
Author

I'd also like to ask you to reconsider disabling # noqa, since flake8 3.0 now supports finer-grained ignores. There's also a --disable-noqa switch to flake8 if someone wants that behaviour.

@habnabit
Copy link
Collaborator

@The-Compiler, sure, lack of fine-grained ignores was the reason it was disabled. Is there a way to disable blanket # noqa comments while still allowing # noqa: E123 comments?

@sigmavirus24
Copy link

@habnabit there is not unfortunately.

@habnabit
Copy link
Collaborator

habnabit commented Jun 28, 2016 via email

@sigmavirus24
Copy link

@habnabit that seems reasonable.

Julian added a commit to Julian/ebb-lint that referenced this issue Nov 26, 2016
Julian added a commit to Julian/ebb-lint that referenced this issue Nov 26, 2016
Julian added a commit to Julian/ebb-lint that referenced this issue Nov 26, 2016
@markrwilliams
Copy link
Member

#9 closes this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants