flake8-jungle

A flake8 plugin to detect bad practices in projects. This plugin is based on flake8-django.

Installation

Install from pip with:

$ pip install flake8-jungle

pre-commit example

  - repo: https://github.com/pycqa/flake8
    rev: 4.0.1
    hooks:
      - id: flake8
        additional_dependencies: ['flake8-jungle==VERSION']
        args: ['--max-condition-complexity=8']

List of Rules

Rule	Description	Configuration
JG02	Do not use exclude attribute in ModelForm, list all items explicitly in fields attribute instead.
JG04	Exceptions should never pass silently, add logging or comment at least.
JG05	Condition is too complex which makes it hard to understand.	--max-condition-complexity
JG06	Function is too long.	--max-function-length
JG07	Model is too long, split it into services, selectors, or utilities.	--max-model-length
JG08	Function or method contains local imports, which should be mostly avoided. If you are trying to fix curcular dependency issues, the design probably has some flaws, you should consider refactoring instead.
JG10	Too much patching in tests. Consider changing your design to utilize Dependency Injection and fakes.	--max-patches-in-test
JG11	Please use structlog and follow the correct logging style: logger.info("snake_case_message.with_dots", key="value").

The following rules are disabled by default:

Rule	Description	Configuration
JG01	The order of the model's inner classes, methods, and fields does not follow the Django Style Guide.
JG03	Avoid using null=True on string-based fields such as CharField and TextField.
JG09	Incorrect logging format, please use the following syntax: logger.info("MESSAGE %(arg1)s", {"arg1": "value1"}).

To enable optional rules you can use the --select parameter. It's default values are: E,F,W,C90.

For example, if you wanted to enable JG10, you could call flake8 in the following way:

flake8 --select=E,F,W,C90,JG,JG10

Testing

flake8-jungle uses pytest for tests. To run them use:

$ poetry install
$ poetry run pytest tests