reef-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mariia Mykhailova <>
Subject RE: Automated code cleanup before releases?
Date Tue, 26 Apr 2016 17:43:47 GMT
These issues don't look like something that should be done as part of release management. Enabling
checkstyle rules is one-time change, not a recurring activity. For license headers, ideally
we also need to come up with some tool to detect this at commit time. I'll file JIRAs for


-----Original Message-----
From: Dongjoon Hyun [] 
Sent: Tuesday, April 26, 2016 10:36 AM
Subject: Re: Automated code cleanup before releases?

For code cleanup, please consider the license header.

- /**
   * Licensed to the Apache Software Foundation (ASF) under one ```

And, if possible, I hope the following CheckStyle rule will be enabled.

<!-- module name="NewlineAtEndOfFile"/--> ```

I sent this email for a record for next release manager.


On Fri, Apr 22, 2016 at 12:04 PM, Dongjoon Hyun <> wrote:

> Oh, I got it.
> +1 for regular Clean-Ups by release managers.
> Dongjoon.
> On Fri, Apr 22, 2016 at 11:39 AM, Markus Weimer <> wrote:
>> On 4/22/2016 11:36 AM, Dongjoon Hyun wrote:
>>> Why don't we clean up the master branch then?
>> Yes, that is what I meant: We clean `master` right before forking off 
>> the release branch. If we make it a routine to file a 
>> release-blocking JIRA to do that, we end up with a clean `master` as part of every
>> Maybe it would be a starter jira issue, too.
>> Maybe. I am a bit hesitant because these commits tend to be very
>> invasive: Lots of lines of code change, lots of files move around. 
>> Hence, while not *technically* challenging, the *coordination* around 
>> it requires a decent feel for the community.
>> Markus
View raw message