Re: add component tag to pull request title / commit comment

[Maximilian Michels]

@Aljoscha: Yes, less space for the commit message if it has to fit
into 70 chars. I'm not sure how important a 70 character limit is
nowadays.

@Davor I think your observation is correct. In Flink we also like to
do tagged commits but enforcing it is almost impossible. Automatic
tagging would be super cool.

On Wed, May 11, 2016 at 4:11 PM, Davor Bonaci  wrote:
> In general, I think this is fine to try, but I think its reliability will
> likely end up being low. There's a human factor involved -- people will
> forget, pull request will evolve in unexpected ways -- all leading to a low
> reliability of this indicator.
>
> I think it would be really awesome to have automatic tagging. Not really on
> the priority list right now, but I'm hopeful we'll have something like that
> in the not-so-distant future.
>
> On Wed, May 11, 2016 at 3:00 AM, Aljoscha Krettek 
> wrote:
>
>> This will, however, also take precious space in the Commit Title. And some
>> commits might not be about only one clear-cut component.
>>
>> On Wed, 11 May 2016 at 11:43 Maximilian Michels  wrote:
>>
>> > +1 I think it makes it easier to see at a glance to which part of Beam
>> > a commit belongs. We could use the Jira components as tags.
>> >
>> > On Wed, May 11, 2016 at 11:09 AM, Jean-Baptiste Onofré 
>> > wrote:
>> > > Hi Manu,
>> > >
>> > > good idea. Theoretically the component in the corresponding Jira should
>> > give
>> > > the information, but for convenience, we could add a "tag" in the
>> commit
>> > > comment.
>> > >
>> > > Regards
>> > > JB
>> > >
>> > >
>> > > On 05/10/2016 06:27 AM, Manu Zhang wrote:
>> > >>
>> > >> Guys,
>> > >>
>> > >> As I've been developing Gearpump runner for Beam, I've closely watched
>> > the
>> > >> pull requests updates for API changes. Sometimes, it turns out changes
>> > are
>> > >> only to be applied to a specific runner after I go through the codes.
>> > >> Could
>> > >> we add a tag in the pull request title / commit comment to mark
>> whether
>> > >> it's API change or Runner specific change, or something else, like,
>> > >> [BEAM-XXX] [Direct-Java] Comments... ?
>> > >>
>> > >> Thanks,
>> > >> Manu Zhang
>> > >>
>> > >
>> > > --
>> > > Jean-Baptiste Onofré
>> > > jbono...@apache.org
>> > > http://blog.nanthrax.net
>> > > Talend - http://www.talend.com
>> >
>>

Re: add component tag to pull request title / commit comment

[Aljoscha Krettek]

This will, however, also take precious space in the Commit Title. And some
commits might not be about only one clear-cut component.

On Wed, 11 May 2016 at 11:43 Maximilian Michels  wrote:

> +1 I think it makes it easier to see at a glance to which part of Beam
> a commit belongs. We could use the Jira components as tags.
>
> On Wed, May 11, 2016 at 11:09 AM, Jean-Baptiste Onofré 
> wrote:
> > Hi Manu,
> >
> > good idea. Theoretically the component in the corresponding Jira should
> give
> > the information, but for convenience, we could add a "tag" in the commit
> > comment.
> >
> > Regards
> > JB
> >
> >
> > On 05/10/2016 06:27 AM, Manu Zhang wrote:
> >>
> >> Guys,
> >>
> >> As I've been developing Gearpump runner for Beam, I've closely watched
> the
> >> pull requests updates for API changes. Sometimes, it turns out changes
> are
> >> only to be applied to a specific runner after I go through the codes.
> >> Could
> >> we add a tag in the pull request title / commit comment to mark whether
> >> it's API change or Runner specific change, or something else, like,
> >> [BEAM-XXX] [Direct-Java] Comments... ?
> >>
> >> Thanks,
> >> Manu Zhang
> >>
> >
> > --
> > Jean-Baptiste Onofré
> > jbono...@apache.org
> > http://blog.nanthrax.net
> > Talend - http://www.talend.com
>

Re: add component tag to pull request title / commit comment

[Jean-Baptiste Onofré]

Hi Manu,

good idea. Theoretically the component in the corresponding Jira should 
give the information, but for convenience, we could add a "tag" in the 
commit comment.


Regards
JB

On 05/10/2016 06:27 AM, Manu Zhang wrote:

Guys,

As I've been developing Gearpump runner for Beam, I've closely watched the
pull requests updates for API changes. Sometimes, it turns out changes are
only to be applied to a specific runner after I go through the codes. Could
we add a tag in the pull request title / commit comment to mark whether
it's API change or Runner specific change, or something else, like,
[BEAM-XXX] [Direct-Java] Comments... ?

Thanks,
Manu Zhang



--
Jean-Baptiste Onofré
jbono...@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com