1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
community.general/test/sanity/pep8
Sloane Hertel 4749b75090 [cloud] ec2_eni: don't modify a network interface without eni_id - fixes #22401 (#22689)
* don't modify an eni without eni_id

fix pep8

remove ec2_eni from pep8 legacy files

remove a couple unused variables in main()

fix the documentation

* fix yaml
2017-03-17 16:26:49 -04:00
..
current-ignore.txt Add pep8 to ansible-test. (#20745) 2017-01-27 12:34:00 -08:00
legacy-files.txt [cloud] ec2_eni: don't modify a network interface without eni_id - fixes #22401 (#22689) 2017-03-17 16:26:49 -04:00
legacy-ignore.txt PEP8 E712 fixes (#21916) 2017-02-24 16:49:43 -06:00
README.md add instructions for running pep8 locally (#21120) 2017-02-13 11:09:50 -08:00
skip.txt refactoring async 2017-02-06 19:32:00 -05:00

PEP 8

PEP 8 style guidelines are enforced by pep8 on all python files in the repository by default.

Current Rule Set

By default all files are tested using the current rule set. All pep8 tests are executed, except those listed in the current ignore list.

Legacy Rule Set

Files which are listed in the legacy file list are tested using the legacy rule set. All pep8 tests are executed, except those listed in the current ignore list or legacy ignore list.

Files listed in the legacy file list which pass the current rule set will result in an error. This is intended to prevent regressions on style guidelines for files which pass the more stringent current rule set.

Skipping Tests

Files listed in the skip list are not tested by pep8.

Removed Files

Files which have been removed from the repository must be removed from the legacy file list and the skip list.

Running Locally

The pep8 check can be run locally with:

./test/runner/ansible-test sanity --test pep8 [file-or-directory-path-to-check] ...