From: John Mulligan <jmulligan@redhat.com>
Date: Tue, 7 May 2024 17:55:56 +0000 (-0400)
Subject: pybind/mgr: remove fix (autopep8) rule from tox.ini
X-Git-Tag: v20.0.0~1618^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=3c3cd0bb119c0258f2e1d2a86adae542231e6fa9;p=ceph.git

pybind/mgr: remove fix (autopep8) rule from tox.ini

Remove the annoying and unloved fix rule from the default envlist of the
pybind/mgr/tox.ini file.

If you don't like this, and want your code formatted automatically, opt
your module into `black` by {format,check}-black rules in tox.ini.
Embrace the much more widely community accepted tool... enjoy the
automatic cookie-cutter formatting... avoid arguing about formatting...
;-)

Signed-off-by: John Mulligan <jmulligan@redhat.com>
---

diff --git a/src/pybind/mgr/tox.ini b/src/pybind/mgr/tox.ini
index 62605e7cc2fe..75ea9971ef32 100644
--- a/src/pybind/mgr/tox.ini
+++ b/src/pybind/mgr/tox.ini
@@ -2,7 +2,6 @@
 envlist =
     py3
     mypy
-    fix
     flake8
     jinjalint
     nooptional
@@ -124,30 +123,6 @@ setenv = {[testenv]setenv}
 deps = {[testenv]deps}
 commands = {[testenv]commands}
 
-[testenv:fix]
-basepython = python3
-deps =
-    autopep8
-modules =
-    alerts \
-    balancer \
-    cephadm \
-    cli_api \
-    crash \
-    devicehealth \
-    diskprediction_local \
-    insights \
-    iostat \
-    nfs \
-    orchestrator \
-    prometheus \
-    rgw \
-    status \
-    telemetry
-commands =
-    python --version
-    autopep8 {[autopep8]addopts} \
-      {posargs:{[testenv:fix]modules}}
 
 [testenv:pylint]
 deps =