Switch to flakehell & wemake python styleguide
This commit is contained in:
parent
81d8d9eb71
commit
ceb159a343
6 changed files with 32 additions and 21 deletions
12
.flake8
12
.flake8
|
@ -1,12 +0,0 @@
|
||||||
[flake8]
|
|
||||||
ignore =
|
|
||||||
# long lines
|
|
||||||
E501,
|
|
||||||
# deprecated rule: https://www.flake8rules.com/rules/W503.html
|
|
||||||
W503,
|
|
||||||
# conflict with black on PEP8 interpretation
|
|
||||||
E203,
|
|
||||||
# class attribute is shadowing a python builtin
|
|
||||||
A003,
|
|
||||||
|
|
||||||
max-complexity = 10
|
|
|
@ -43,10 +43,10 @@ repos:
|
||||||
rev: v1.4.11
|
rev: v1.4.11
|
||||||
hooks:
|
hooks:
|
||||||
- id: djhtml
|
- id: djhtml
|
||||||
- repo: https://github.com/pycqa/flake8
|
- repo: https://github.com/mcarans/flakehell
|
||||||
rev: 4.0.1
|
rev: 1b84f4dd6c16232b5c0c6206511427676ab55f5b
|
||||||
hooks:
|
hooks:
|
||||||
- id: flake8
|
- id: flakehell
|
||||||
additional_dependencies:
|
additional_dependencies:
|
||||||
- flake8-annotations-complexity
|
- flake8-annotations-complexity
|
||||||
- flake8-builtins
|
- flake8-builtins
|
||||||
|
@ -56,6 +56,7 @@ repos:
|
||||||
- flake8-noqa
|
- flake8-noqa
|
||||||
- flake8-pytest-style
|
- flake8-pytest-style
|
||||||
- flake8-pyi
|
- flake8-pyi
|
||||||
|
- wemake-python-styleguide
|
||||||
- repo: https://github.com/pre-commit/mirrors-prettier
|
- repo: https://github.com/pre-commit/mirrors-prettier
|
||||||
rev: v2.5.1
|
rev: v2.5.1
|
||||||
hooks:
|
hooks:
|
||||||
|
|
|
@ -81,6 +81,30 @@ ignore_missing_imports = true
|
||||||
[tool.django-stubs]
|
[tool.django-stubs]
|
||||||
django_settings_module = "blog.settings"
|
django_settings_module = "blog.settings"
|
||||||
|
|
||||||
|
[tool.flakehell]
|
||||||
|
max_complexity = 10
|
||||||
|
format = "grouped"
|
||||||
|
|
||||||
|
[tool.flakehell.plugins]
|
||||||
|
"flake8-*" = [
|
||||||
|
"+*",
|
||||||
|
# long lines
|
||||||
|
"-E501",
|
||||||
|
# conflict with black on PEP8 interpretation
|
||||||
|
"-E203",
|
||||||
|
# deprecated rule: https://www.flake8rules.com/rules/W503.html
|
||||||
|
"-W503",
|
||||||
|
]
|
||||||
|
flake8-builtins = ["-A003"] # class attribute is shadowing a python builtin
|
||||||
|
flake8-quotes = ["-Q000"] # found double quotes, conflict with black
|
||||||
|
flake8-commas = ["-C812"] # missing trailing comma, conflict with black
|
||||||
|
flake8-docstrings = ["-D1??"] # missing docstring
|
||||||
|
flake8-rst-docstrings = ["-*"]
|
||||||
|
|
||||||
|
[tool.flakehell.exceptions."**/migrations/*"]
|
||||||
|
|
||||||
|
[tool.flakehell.exceptions."**/tests/*"]
|
||||||
|
flake8-bandit = ["-S101"] # Use of assert detected.
|
||||||
|
|
||||||
[build-system]
|
[build-system]
|
||||||
requires = ["poetry-core>=1.0.0"]
|
requires = ["poetry-core>=1.0.0"]
|
||||||
|
|
|
@ -34,9 +34,7 @@ class CompleteFeed(BaseFeed):
|
||||||
|
|
||||||
|
|
||||||
class TagFeed(BaseFeed):
|
class TagFeed(BaseFeed):
|
||||||
def get_object( # type: ignore[override]
|
def get_object(self, request: WSGIRequest, *args, **kwargs) -> Tag: # type: ignore[override]
|
||||||
self, request: WSGIRequest, *args, **kwargs
|
|
||||||
) -> Tag:
|
|
||||||
return Tag.objects.get(slug=kwargs.get("slug"))
|
return Tag.objects.get(slug=kwargs.get("slug"))
|
||||||
|
|
||||||
def title(self, tag: Tag) -> str:
|
def title(self, tag: Tag) -> str:
|
||||||
|
|
|
@ -41,7 +41,7 @@ class AttachmentAdmin(admin.ModelAdmin):
|
||||||
def processed_file_url(self, instance):
|
def processed_file_url(self, instance):
|
||||||
if instance.processed_file:
|
if instance.processed_file:
|
||||||
return format_html(
|
return format_html(
|
||||||
'{} <a class="copy-button" data-to-copy="{}" href="#">📋</a>',
|
'{0} <a class="copy-button" data-to-copy="{1}" href="#">📋</a>',
|
||||||
instance.processed_file.url,
|
instance.processed_file.url,
|
||||||
instance.processed_file.url,
|
instance.processed_file.url,
|
||||||
)
|
)
|
||||||
|
@ -50,7 +50,7 @@ class AttachmentAdmin(admin.ModelAdmin):
|
||||||
def original_file_url(self, instance):
|
def original_file_url(self, instance):
|
||||||
if instance.original_file:
|
if instance.original_file:
|
||||||
return format_html(
|
return format_html(
|
||||||
'{} <a class="copy-button" data-to-copy="{}" href="#">📋</a>',
|
'{0} <a class="copy-button" data-to-copy="{1}" href="#">📋</a>',
|
||||||
instance.original_file.url,
|
instance.original_file.url,
|
||||||
instance.original_file.url,
|
instance.original_file.url,
|
||||||
)
|
)
|
||||||
|
|
|
@ -5,7 +5,7 @@ import sys
|
||||||
|
|
||||||
|
|
||||||
def main():
|
def main():
|
||||||
"""Run administrative tasks."""
|
"""Run administrative tasks.""" # noqa: DAR401
|
||||||
os.environ.setdefault("DJANGO_SETTINGS_MODULE", "blog.settings")
|
os.environ.setdefault("DJANGO_SETTINGS_MODULE", "blog.settings")
|
||||||
try:
|
try:
|
||||||
from django.core.management import execute_from_command_line
|
from django.core.management import execute_from_command_line
|
||||||
|
|
Reference in a new issue