You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@druid.apache.org by Roman Leventov <le...@apache.org> on 2019/03/14 21:36:10 UTC
About [tags] in issue and PR headers
1) Do other people want proposal issues to have [PROPOSAL] in their names,
despite they also have a corresponding tag? Maybe we can at least make it
not caps?
2) Would be nice to teach bot to visit PRs and issues from non-committers
and remove "[tag name]" parts from their titles and assign corresponding
tags.
Re: About [tags] in issue and PR headers
Posted by Gian Merlino <gi...@gmail.com>.
I’d also be ok with ditching them all except “Backport”.
Gian
> On Mar 15, 2019, at 15:36, Jonathan Wei <jo...@apache.org> wrote:
>
> I don't mind the [TAG] in the titles, but I think github labels are better,
> and I'm fine with removing the [TAG]s.
>
> [Backport] would be the only tag I feel we should keep, just to
> disambiguate between the backport and the original PR.
>
>> On Fri, Mar 15, 2019 at 11:26 AM Gian Merlino <gi...@apache.org> wrote:
>>
>> I do think that other than a couple key ones like proposal and backport, we
>> should encourage people to not put tags (or issue #s) in PR titles.
>>
>>> On Thu, Mar 14, 2019 at 7:56 PM Gian Merlino <gi...@apache.org> wrote:
>>>
>>> Personally I use email as my main interface to see what's new in github,
>>> and it doesn't show labels, and I think being able to pick out proposals
>>> and backports easily is useful. So I like the tags. But, not so much
>> that I
>>> would fight to keep them if consensus is going the other direction.
>>>
>>> On Thu, Mar 14, 2019 at 2:59 PM Clint Wylie <cl...@imply.io>
>> wrote:
>>>
>>>> I find [PROPOSAL] and all other title tags redundant and prefer the
>> github
>>>> labels since they are easier to use for searching, and would prefer the
>>>> tags in the title were not there. I guess I don't mind [Backport] as
>> much
>>>> because it makes it really stand out that it's a PR for backporting for
>>>> release, but it is also probably not necessary if we just want to get
>> rid
>>>> of them entirely. I feel the same about putting the issue number in the
>> PR
>>>> title, which is also pretty worthless since it doesn't link it.
>>>>
>>>> A bot to automatically do that might be nice, +1. I wonder how much work
>>>> it
>>>> would be to examine which paths are modified by a PR to attempt to
>>>> automatically add 'Area' labels too, but that might be overkill.
>>>>
>>>> On Thu, Mar 14, 2019 at 2:36 PM Roman Leventov <le...@apache.org>
>>>> wrote:
>>>>
>>>>> 1) Do other people want proposal issues to have [PROPOSAL] in their
>>>> names,
>>>>> despite they also have a corresponding tag? Maybe we can at least make
>>>> it
>>>>> not caps?
>>>>>
>>>>> 2) Would be nice to teach bot to visit PRs and issues from
>>>> non-committers
>>>>> and remove "[tag name]" parts from their titles and assign
>> corresponding
>>>>> tags.
>>>>>
>>>>
>>>
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@druid.apache.org
For additional commands, e-mail: dev-help@druid.apache.org
Re: About [tags] in issue and PR headers
Posted by Jonathan Wei <jo...@apache.org>.
I don't mind the [TAG] in the titles, but I think github labels are better,
and I'm fine with removing the [TAG]s.
[Backport] would be the only tag I feel we should keep, just to
disambiguate between the backport and the original PR.
On Fri, Mar 15, 2019 at 11:26 AM Gian Merlino <gi...@apache.org> wrote:
> I do think that other than a couple key ones like proposal and backport, we
> should encourage people to not put tags (or issue #s) in PR titles.
>
> On Thu, Mar 14, 2019 at 7:56 PM Gian Merlino <gi...@apache.org> wrote:
>
> > Personally I use email as my main interface to see what's new in github,
> > and it doesn't show labels, and I think being able to pick out proposals
> > and backports easily is useful. So I like the tags. But, not so much
> that I
> > would fight to keep them if consensus is going the other direction.
> >
> > On Thu, Mar 14, 2019 at 2:59 PM Clint Wylie <cl...@imply.io>
> wrote:
> >
> >> I find [PROPOSAL] and all other title tags redundant and prefer the
> github
> >> labels since they are easier to use for searching, and would prefer the
> >> tags in the title were not there. I guess I don't mind [Backport] as
> much
> >> because it makes it really stand out that it's a PR for backporting for
> >> release, but it is also probably not necessary if we just want to get
> rid
> >> of them entirely. I feel the same about putting the issue number in the
> PR
> >> title, which is also pretty worthless since it doesn't link it.
> >>
> >> A bot to automatically do that might be nice, +1. I wonder how much work
> >> it
> >> would be to examine which paths are modified by a PR to attempt to
> >> automatically add 'Area' labels too, but that might be overkill.
> >>
> >> On Thu, Mar 14, 2019 at 2:36 PM Roman Leventov <le...@apache.org>
> >> wrote:
> >>
> >> > 1) Do other people want proposal issues to have [PROPOSAL] in their
> >> names,
> >> > despite they also have a corresponding tag? Maybe we can at least make
> >> it
> >> > not caps?
> >> >
> >> > 2) Would be nice to teach bot to visit PRs and issues from
> >> non-committers
> >> > and remove "[tag name]" parts from their titles and assign
> corresponding
> >> > tags.
> >> >
> >>
> >
>
Re: About [tags] in issue and PR headers
Posted by Gian Merlino <gi...@apache.org>.
I do think that other than a couple key ones like proposal and backport, we
should encourage people to not put tags (or issue #s) in PR titles.
On Thu, Mar 14, 2019 at 7:56 PM Gian Merlino <gi...@apache.org> wrote:
> Personally I use email as my main interface to see what's new in github,
> and it doesn't show labels, and I think being able to pick out proposals
> and backports easily is useful. So I like the tags. But, not so much that I
> would fight to keep them if consensus is going the other direction.
>
> On Thu, Mar 14, 2019 at 2:59 PM Clint Wylie <cl...@imply.io> wrote:
>
>> I find [PROPOSAL] and all other title tags redundant and prefer the github
>> labels since they are easier to use for searching, and would prefer the
>> tags in the title were not there. I guess I don't mind [Backport] as much
>> because it makes it really stand out that it's a PR for backporting for
>> release, but it is also probably not necessary if we just want to get rid
>> of them entirely. I feel the same about putting the issue number in the PR
>> title, which is also pretty worthless since it doesn't link it.
>>
>> A bot to automatically do that might be nice, +1. I wonder how much work
>> it
>> would be to examine which paths are modified by a PR to attempt to
>> automatically add 'Area' labels too, but that might be overkill.
>>
>> On Thu, Mar 14, 2019 at 2:36 PM Roman Leventov <le...@apache.org>
>> wrote:
>>
>> > 1) Do other people want proposal issues to have [PROPOSAL] in their
>> names,
>> > despite they also have a corresponding tag? Maybe we can at least make
>> it
>> > not caps?
>> >
>> > 2) Would be nice to teach bot to visit PRs and issues from
>> non-committers
>> > and remove "[tag name]" parts from their titles and assign corresponding
>> > tags.
>> >
>>
>
Re: About [tags] in issue and PR headers
Posted by Gian Merlino <gi...@apache.org>.
Personally I use email as my main interface to see what's new in github,
and it doesn't show labels, and I think being able to pick out proposals
and backports easily is useful. So I like the tags. But, not so much that I
would fight to keep them if consensus is going the other direction.
On Thu, Mar 14, 2019 at 2:59 PM Clint Wylie <cl...@imply.io> wrote:
> I find [PROPOSAL] and all other title tags redundant and prefer the github
> labels since they are easier to use for searching, and would prefer the
> tags in the title were not there. I guess I don't mind [Backport] as much
> because it makes it really stand out that it's a PR for backporting for
> release, but it is also probably not necessary if we just want to get rid
> of them entirely. I feel the same about putting the issue number in the PR
> title, which is also pretty worthless since it doesn't link it.
>
> A bot to automatically do that might be nice, +1. I wonder how much work it
> would be to examine which paths are modified by a PR to attempt to
> automatically add 'Area' labels too, but that might be overkill.
>
> On Thu, Mar 14, 2019 at 2:36 PM Roman Leventov <le...@apache.org>
> wrote:
>
> > 1) Do other people want proposal issues to have [PROPOSAL] in their
> names,
> > despite they also have a corresponding tag? Maybe we can at least make it
> > not caps?
> >
> > 2) Would be nice to teach bot to visit PRs and issues from non-committers
> > and remove "[tag name]" parts from their titles and assign corresponding
> > tags.
> >
>
Re: About [tags] in issue and PR headers
Posted by Clint Wylie <cl...@imply.io>.
I find [PROPOSAL] and all other title tags redundant and prefer the github
labels since they are easier to use for searching, and would prefer the
tags in the title were not there. I guess I don't mind [Backport] as much
because it makes it really stand out that it's a PR for backporting for
release, but it is also probably not necessary if we just want to get rid
of them entirely. I feel the same about putting the issue number in the PR
title, which is also pretty worthless since it doesn't link it.
A bot to automatically do that might be nice, +1. I wonder how much work it
would be to examine which paths are modified by a PR to attempt to
automatically add 'Area' labels too, but that might be overkill.
On Thu, Mar 14, 2019 at 2:36 PM Roman Leventov <le...@apache.org> wrote:
> 1) Do other people want proposal issues to have [PROPOSAL] in their names,
> despite they also have a corresponding tag? Maybe we can at least make it
> not caps?
>
> 2) Would be nice to teach bot to visit PRs and issues from non-committers
> and remove "[tag name]" parts from their titles and assign corresponding
> tags.
>