[openstack-dev] Garbage patches for simple typo fixes

Matt Riedemann mriedemos at gmail.com
Fri Sep 22 15:00:19 UTC 2017


On 9/22/2017 9:59 AM, Matt Riedemann wrote:
> Figuring out what is useless or not is probably not worth the effort 
> here. We already skip long running tempest dsvm jobs in certain patches, 
> like with docs or unit test only changes. Updating a code comment in 
> code isn't going to catch that.
> 
> And it's perfectly valid to have useful single-line code changes 
> (although if it's a bug there should be a test too). Or multi-line 
> changes that are just adding comments to code.
> 
> Plus most people are probably not going to review something until CI has 
> voted on it anyway, at least when you have the number of open reviews 
> that some projects, like nova, has.
> 
> So I think this is a non-starter.

To summarize, this is a social issue, not a technical issue.

-- 

Thanks,

Matt



More information about the OpenStack-dev mailing list