A flake8 plugin to detect bad practices in projects. This plugin is based on flake8-django.
Install from pip with:
$ pip install flake8-jungle
- repo: https://github.com/pycqa/flake8
rev: 4.0.1
hooks:
- id: flake8
additional_dependencies: ['flake8-jungle==VERSION']
args: ['--max-condition-complexity=8']
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
flake8-jungle uses pytest for tests. To run them use:
$ poetry install
$ poetry run pytest tests