cp-ansible icon indicating copy to clipboard operation
cp-ansible copied to clipboard

Fix pep8 issues, add 2.15 and 2.16 sanity ignore

Open chynasan opened this issue 4 months ago • 1 comments

Description

Updated spacing and corrected ansible-test sanity pep8 line-too-long errors in plugins/modules/kafka_connectors.py. Added tests/sanity/ignore-2.15.txt and tests/sanity/ignore-2.16.txt to ensure ignores are up-to-date for versions tested against. Added an ignore to .ansible-lint for the molecule directory.

Included 2.14 changes from #1557

Type of change

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

How Has This Been Tested?

Ran ansible-test sanity --docker default in venv for 2.15 and 2.16 to confirm pep8 and other logging is clean.

Checklist:

  • [x] Any variable/code changes have been validated to be backwards compatible (doesn't break upgrade)
  • [x] I have added tests that prove my fix is effective or that my feature works
  • [x] If required, I have ensured the changes can be discovered by cp-ansible discovery codebase
  • [x] My code follows the style guidelines of this project
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [x] Any dependent changes have been merged and published in downstream modules

Regards, Ansible PE

chynasan avatar Feb 19 '24 18:02 chynasan