Skip to content

Latest commit

 

History

History
129 lines (90 loc) · 4.6 KB

plugin_handling.rst

File metadata and controls

129 lines (90 loc) · 4.6 KB

Plugin Handling

Plugin Management

3.0 added support for two other plugins besides those which define new checks. It now supports:

  • extra checks
  • alternative report formatters
  • listeners to auto-correct violations of checks

To facilitate this, needed a more mature way of managing plugins. Thus, we developed the ~flake8.plugins.manager.PluginManager which accepts a namespace and will load the plugins for that namespace. A ~flake8.plugins.manager.PluginManager creates and manages many ~flake8.plugins.manager.Plugin instances.

A ~flake8.plugins.manager.Plugin lazily loads the underlying entry-point provided by setuptools. The entry-point will be loaded either by calling ~flake8.plugins.manager.Plugin.load_plugin or accessing the plugin attribute. We also use this abstraction to retrieve options that the plugin wishes to register and parse.

The only public method the ~flake8.plugins.manager.PluginManager provides is ~flake8.plugins.manager.PluginManager.map. This will accept a function (or other callable) and call it with each plugin as the first parameter.

We build atop the ~flake8.plugins.manager.PluginManager with the ~flake8.plugins.manager.PluginTypeManager. It is expected that users of the ~flake8.plugins.manager.PluginTypeManager will subclass it and specify the namespace, e.g.,

class ExamplePluginType(flake8.plugin.manager.PluginTypeManager):
    namespace = 'example-plugins'

This provides a few extra methods via the ~flake8.plugins.manager.PluginManager's map method.

Finally, we create three classes of plugins:

  • ~flake8.plugins.manager.Checkers
  • ~flake8.plugins.manager.Listeners
  • ~flake8.plugins.manager.ReportFormatters

These are used to interact with each of the types of plugins individually.

Note

Our inspiration for our plugin handling comes from the author's extensive experience with stevedore.

Notifying Listener Plugins

One of the interesting challenges with allowing plugins to be notified each time an error or warning is emitted by a checker is finding listeners quickly and efficiently. It makes sense to allow a listener to listen for a certain class of warnings or just a specific warning. Hence, we need to allow all plugins that listen to a specific warning or class to be notified. For example, someone might register a listener for E1 and another for E111 if E111 is triggered by the code, both listeners should be notified. If E112 is returned, then only E1 (and any other listeners) would be notified.

To implement this goal, we needed an object to store listeners in that would allow for efficient look up - a Trie (or Prefix Tree). Given that none of the existing packages on PyPI allowed for storing data on each node of the trie, it was left up to write our own as ~flake8.plugins._trie.Trie. On top of that we layer our ~flake8.plugins.notifier.Notifier class.

Now when receives an error or warning, we can easily call the ~flake8.plugins.notifier.Notifier.notify method and let plugins act on that knowledge.

Default Plugins

Finally, has always provided its own plugin shim for Pyflakes. As part of that we carry our own shim in-tree and now store that in flake8.plugins.pyflakes.

also registers plugins for pep8. Each check in pep8 requires different parameters and it cannot easily be shimmed together like Pyflakes was. As such, plugins have a concept of a "group". If you look at our setup.py you will see that we register pep8 checks roughly like so:

pep8.<check-name> = pep8:<check-name>

We do this to identify that <check-name>> is part of a group. This also enables us to special-case how we handle reporting those checks. Instead of reporting each check in the --version output, we report pep8 and check pep8 the module for a __version__ attribute. We only report it once to avoid confusing users.

API Documentation

flake8.plugins.manager.PluginManager

flake8.plugins.manager.Plugin

flake8.plugins.manager.PluginTypeManager

flake8.plugins.manager.Checkers

flake8.plugins.manager.Listeners

flake8.plugins.manager.ReportFormatters

flake8.plugins.notifier.Notifier

flake8.plugins._trie.Trie