launchpad-reviewers team mailing list archive

[Colin Watson <mp+412129@xxxxxxxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/lpcraft:restore-setup-cfg-fmt into lpcraft:main.

Commit message:
Upgrade to setup-cfg-fmt 1.20.0

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/lpcraft/+git/lpcraft/+merge/412129

This is much faster thanks to improvements in `identify`, so we can run it as a `pre-commit` hook again.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/lpcraft:restore-setup-cfg-fmt into lpcraft:main.

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 7de2d0e..f1856e7 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -23,3 +23,7 @@ repos:
     rev: 21.10b0
     hooks:
     -   id: black
+-   repo: https://github.com/asottile/setup-cfg-fmt
+    rev: v1.20.0
+    hooks:
+    -   id: setup-cfg-fmt
diff --git a/tox.ini b/tox.ini
index bc6d96b..4bba8cc 100644
--- a/tox.ini
+++ b/tox.ini
@@ -46,12 +46,3 @@ commands =
     coverage run -m pytest
     coverage html
     coverage report -m  --fail-under=80
-
-[testenv:setup-cfg-fmt]
-basepython =
-    python3.8
-deps =
-    setup-cfg-fmt
-skip_install = true
-commands =
-    setup-cfg-fmt setup.cfg