Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-02-03 Thread Joe Gordon
On Thu, Jan 30, 2014 at 2:06 AM, Daniel P. Berrange wrote: > On Wed, Jan 29, 2014 at 01:22:59PM -0500, Joe Gordon wrote: >> On Tue, Jan 28, 2014 at 4:45 AM, John Garbutt wrote: >> > On 27 January 2014 10:10, Daniel P. Berrange wrote: >> >> On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wro

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-30 Thread Daniel P. Berrange
On Wed, Jan 29, 2014 at 01:22:59PM -0500, Joe Gordon wrote: > On Tue, Jan 28, 2014 at 4:45 AM, John Garbutt wrote: > > On 27 January 2014 10:10, Daniel P. Berrange wrote: > >> On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wrote: > >>> On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange >

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-29 Thread Andreas Jaeger
On 01/29/2014 07:22 PM, Joe Gordon wrote: > On Tue, Jan 28, 2014 at 4:45 AM, John Garbutt wrote: >> On 27 January 2014 10:10, Daniel P. Berrange wrote: >>> On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wrote: On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange wrote: >

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-29 Thread Joe Gordon
On Tue, Jan 28, 2014 at 4:45 AM, John Garbutt wrote: > On 27 January 2014 10:10, Daniel P. Berrange wrote: >> On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wrote: >>> On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange >>> wrote: >>> >>> > Periodically I've seen people submit big coding s

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-28 Thread John Garbutt
On 27 January 2014 10:10, Daniel P. Berrange wrote: > On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wrote: >> On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange >> wrote: >> >> > Periodically I've seen people submit big coding style cleanups to Nova >> > code. These are typically all good

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-27 Thread Daniel P. Berrange
On Fri, Jan 24, 2014 at 11:42:54AM -0500, Joe Gordon wrote: > On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange > wrote: > > > Periodically I've seen people submit big coding style cleanups to Nova > > code. These are typically all good ideas / beneficial, however, I have > > rarely (perhaps ev

Re: [openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-24 Thread Joe Gordon
On Fri, Jan 24, 2014 at 7:24 AM, Daniel P. Berrange wrote: > Periodically I've seen people submit big coding style cleanups to Nova > code. These are typically all good ideas / beneficial, however, I have > rarely (perhaps even never?) seen the changes accompanied by new hacking > check rules. > >

[openstack-dev] Nova style cleanups with associated hacking check addition

2014-01-24 Thread Daniel P. Berrange
Periodically I've seen people submit big coding style cleanups to Nova code. These are typically all good ideas / beneficial, however, I have rarely (perhaps even never?) seen the changes accompanied by new hacking check rules. The problem with not having a hacking check added *in the same commit*