Sean Mooney
8eb3a5a4b2
Replace autopep8 with ruff-format in pre-commit. Add G and LOG rule-sets to ruff lint so logging-format violations (e.g. G003 string concatenation in log calls) are caught without hacking. Add external = ["H"] so ruff recognises H-codes in noqa comments. Add [tool.ruff.format] with quote-style = "preserve" to avoid mass quote conversion on first run (oslo.utils pattern). Remove the local cyborg/hacking/ plugin (M310/M322/M336/M340 checks) and its test file; register only the upstream hacking package (rev 8.0.0) via pre-commit. Simplify [flake8] to select = H only, matching oslo.utils. Remove [flake8:local-plugins]. Drop bandit: remove tox bandit env, cyborg-tox-bandit Zuul job and its check reference, and bandit from test-requirements.txt. Hacking is also removed from test-requirements.txt; it is now consumed solely via the pre-commit hook. Rewrite tox pep8 env to run `pre-commit run -a` (oslo.utils pattern); remove fast8 env and tools/flake8wrap.sh. Remove autopep8 mention from HACKING.rst. Apply ruff-format to all 214 reformatted source files as part of this migration commit. Generated-By: claude-code sonnet 4.6 Change-Id: Id70f611dd571e156264838d4054ff62577028671 Signed-off-by: Sean Mooney <work@seanmooney.info>
686 B
686 B
cyborg Style Commandments
If you haven't already done so read the OpenStack Style Commandments https://docs.openstack.org/hacking/latest/
Before you commit your code run tox against your patch using the command.
tox .
If any of the tests fail correct the error and try again.
Once you submit a patch integration tests will run and those may fail, -1'ing your patch you can make a gerrit comment 'recheck ci' if you have reviewed the logs from the jobs by clicking on the job name in gerrit and concluded that the failure was spurious or otherwise not related to your patch. If problems persist contact people on #openstack-cyborg or #openstack-infra.