On Thu, Nov 16, 2017 at 04:42:11PM -0600, Sean McGinnis wrote: > On Thu, Nov 16, 2017 at 01:41:56PM -0600, Sean McGinnis wrote: > > On Thu, Nov 16, 2017 at 12:58:38PM -0600, Matt Riedemann wrote: > > > >> > > > >>So we'll probably need to change the definition of the nova-specs py27 > > > >>job to run these linter type tests in the pep8 run. > > > >> > > > > > > [snip] > > > > > > > > The right answer is probably to just switch those to run doc8 against the > > specs, since it appears the "tests" we have in there just try to do what > > that > > tools is for. > > > > Working on a cinder-specs patch to do that now. > > > > I think this is the approach we will likely want: > > https://review.openstack.org/#/c/520823/ >
I don't have time to look into it right now, but we probably shouldn't run openstack-tox-py27 against these repos in the first place, especially after this change. It is just wasting infra resources to literally do nothing. We are not running pep8 jobs against these repos, which we probably should do now that that will actually catch RST formatting issues in newly proposed specs. __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
