aboutsummaryrefslogtreecommitdiffstats
path: root/python
diff options
context:
space:
mode:
authorBryan Newbold <bnewbold@archive.org>2021-10-27 18:50:42 -0700
committerBryan Newbold <bnewbold@archive.org>2021-10-27 18:50:42 -0700
commitee32fa0fba6995f98d2301fdf66ef39229ff49e1 (patch)
treeed861fb2119c91fbe7ab391cf301457f144a31bb /python
parent826c7538e091fac14d987a3cd654975da964e240 (diff)
downloadsandcrawler-ee32fa0fba6995f98d2301fdf66ef39229ff49e1.tar.gz
sandcrawler-ee32fa0fba6995f98d2301fdf66ef39229ff49e1.zip
toolchain config updates
Diffstat (limited to 'python')
-rw-r--r--python/.flake87
-rw-r--r--python/Makefile3
-rw-r--r--python/pyproject.toml6
3 files changed, 6 insertions, 10 deletions
diff --git a/python/.flake8 b/python/.flake8
index 4dd17b7..9c9aabe 100644
--- a/python/.flake8
+++ b/python/.flake8
@@ -4,14 +4,15 @@ select = C,E,F,W,ANN
# ANN101 is annotation on 'self' (why would that be wanted?)
# ANN204 is annotation on '__init__()'
# E265,E266 are restrictions on comments ('#')
-# E501 is line-too-long, which we enforce with yapf
+# E501 is line-too-long, which we enforce with black
+# W503,E203 are allowed by black
# TODO: C901 is complexity, should be re-enabled at some point
-# TODO: E251,W504,E126,E125,W503 are unfortunate yapf behaviors; could try to work around them?
-ignore = ANN003,ANN101,ANN204,E265,E266,E501,C901,E251,W504,E126,E125,W503
+ignore = ANN003,ANN101,ANN204,E265,E266,E501,C901,W503,E203
per-file-ignores =
sandcrawler/__init__.py: F401
sandcrawler/ia.py: E402
tests/*.py: ANN201,ANN001,F403,F405
+ # TODO: add more annotations to CLI scripts
*_tool.py,sandcrawler_worker.py: ANN201,ANN001,ANN202,ANN206,ANN205,F403,F405
scripts: ANN201,ANN001,ANN202,ANN206,ANN205
exclude = .git,__pycache__,.venv,scripts/
diff --git a/python/Makefile b/python/Makefile
index 4cef708..940a7eb 100644
--- a/python/Makefile
+++ b/python/Makefile
@@ -17,12 +17,11 @@ lint: ## Run lints (eg, flake8, mypy)
pipenv run flake8 . --exit-zero
pipenv run isort -q -c . || true
pipenv run mypy *.py sandcrawler/ tests/ --ignore-missing-imports
- pipenv run pylint --rcfile=.pylintrc -E --jobs=4 sandcrawler tests/*.py *.py
.PHONY: fmt
fmt: ## Run code formating on all source code
pipenv run isort --atomic .
- pipenv run yapf -p -i -r sandcrawler tests scripts *.py
+ pipenv run black --line-length 96 sandcrawler/ tests/ scripts/ *.py
.PHONY: test
test: ## Run all tests and lints
diff --git a/python/pyproject.toml b/python/pyproject.toml
index e85b234..2cef007 100644
--- a/python/pyproject.toml
+++ b/python/pyproject.toml
@@ -2,10 +2,6 @@
requires = ["setuptools", "wheel"]
build-backend = "setuptools.build_meta:__legacy__"
-[tool.yapf]
-COLUMN_LIMIT = 96
-INDENT_DICTIONARY_VALUE=true
-ALLOW_SPLIT_BEFORE_DICT_VALUE=false
-
[tool.isort]
+profile = "black"
line_length = 96