---- On Mon, 16 Dec 2019 13:55:52 -0600 Ben Nemec <openstack@nemebean.com> wrote ----
On 12/14/19 10:29 AM, Ghanshyam Mann wrote:
* I pushed the patches on specs repo to cleanup py2 specific requirement or tox or zuul changes. many projects were running 'openstack-tox-py27' which I have changed to 'openstack-tox-pep8' instead of moving them to py3. pep8 job is enough for Specs repo.
Are you sure? I believe some specs repos are using unit tests to validate that all of the sections of the spec are included (among other things)[0]. I don't think the pep8 job will cover those.
pep8 job keep running the unit tests for many of them via explicit pep8 tox to run the unit tests[1] or not defining the env itself. I can move them to py3 job to be explicitly readable. But for few spec repo like keystone-specs pep8 would not cover the unit tests as pep8 is defined there as pep8 checks only. I will move py2 jobs to py3 job there. Thanks for monitor those. [1] https://opendev.org/openstack/nova-specs/src/commit/36b1b7fbf52360c5e7e335c7... -gmann
0: https://opendev.org/openstack/nova-specs/src/branch/master/tests/test_titles...