From b7a5411d8b8362e369a0bb80263ed677b31eddfb Mon Sep 17 00:00:00 2001 From: Matt Clay Date: Wed, 19 Mar 2025 18:17:01 -0700 Subject: [PATCH] ansible-test - Improve pep8 compat with black (#84867) --- changelogs/fragments/ansible-test-update.yml | 1 + .../_util/controller/sanity/pep8/current-ignore.txt | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/changelogs/fragments/ansible-test-update.yml b/changelogs/fragments/ansible-test-update.yml index 3200457c814..ed58e33562a 100644 --- a/changelogs/fragments/ansible-test-update.yml +++ b/changelogs/fragments/ansible-test-update.yml @@ -4,3 +4,4 @@ minor_changes: - ansible-test - Disable the ``deprecated-`` prefixed ``pylint`` rules as their results vary by Python version. - ansible-test - Update the ``base`` and ``default`` containers. - ansible-test - Update sanity test requirements to latest available versions. + - ansible-test - Disable the ``pep8`` sanity test rules ``E701`` and ``E704`` to improve compatibility with ``black``. diff --git a/test/lib/ansible_test/_util/controller/sanity/pep8/current-ignore.txt b/test/lib/ansible_test/_util/controller/sanity/pep8/current-ignore.txt index 4d1de69240f..8fd5224b284 100644 --- a/test/lib/ansible_test/_util/controller/sanity/pep8/current-ignore.txt +++ b/test/lib/ansible_test/_util/controller/sanity/pep8/current-ignore.txt @@ -7,3 +7,7 @@ E741 # Unfortunately this means it also conflicts with the output from `black`. # See: https://github.com/PyCQA/pycodestyle/issues/373 E203 + +# The "multiple statements on one line" rules conflict with `black` for no-impl class and method defs. +E701 +E704