Current pep8 test / gates do not actually do pep8, rename to validate

Registered by Borne Mace

Currently the pep8 test does no pep8 testing. It verifies that json / yaml files can be loaded without error and that dockerfiles have appropriate maintainer entries. Suggest renaming the test to validate-contents or something similar / more accurate.

Blueprint information

Status:
Complete
Approver:
Steven Dake
Priority:
Medium
Drafter:
Borne Mace
Direction:
Approved
Assignee:
Borne Mace
Definition:
Approved
Series goal:
Accepted for liberty
Implementation:
Implemented
Milestone target:
milestone icon liberty-2
Started by
Borne Mace
Completed by
Borne Mace

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/rename-pep8-validate,n,z

Addressed by: https://review.openstack.org/198922
    Added additional tox test for json / yaml / link / dockerfile validation. This currently mimics the existing pep8 test which will be updated to run flake8

Addressed by: https://review.openstack.org/203703
    Removed all validate-contents tests from pep8 test

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.