AW: Confirmed bug labels

Matthias St. Pierre Matthias.St.Pierre at ncp-e.com
Tue Oct 29 11:57:23 UTC 2019


Another idea just occurred to me: we could join the 'closed: *' labels with the 'triaged: *' labels:

     triaged: duplicate
     triaged: not a bug
     triaged: wont fix


On 29.10.19 12:53, Matthias St. Pierre wrote:
>
>
> On 29.10.19 12:41, Matt Caswell wrote:
>>
>> On 29/10/2019 11:34, Dr. Matthias St. Pierre wrote:
>>> A similar problem applies to 'issue: feature request'.  Just having a 'confirmed' label for bugs
>>> wouldn't help in that case.
>>>
>>> So what do you think about adding a new 'triaged: *' family of labels, in addition to 'issue: *'?
>>>
>>>     'triaged: bug'
>>>     'triaged: feature'
>>>     etc.
>>>
>>> If this seems too verbose, then we could just omit the triaged prefix:
>>>
>>>     'bug'
>>>     'feature'
>>>     etc.
>> Yes, this makes sense to me (and I prefer the more verbose versions).
>> Should we remove the reporter label once its been triaged? It would be
>> quite confusing if you had both the labels "issue: bug report" *and*
>> "triaged: feature" (in the cases where someone reports something as a
>> bug, but we see it as a feature request).
>
> I agree with you that it should be removed.
>
>
> BTW: That's a similar question than your recent question whether
> 'approval: done' should be removed when the 'ready to merge'
> label is added. After sleeping a night over it, I would prefer if
> the former were removed. If we would add the 'approval: ' prefix,
> then it would be obvious why it makes sense:
>
>     'approval: review pending'
>     'approval: omc review pending'
>     'approval: done'
>
>     ... 24h grace period ...
>
>     'approval: ready to merge'
>
> The transition diagram would be much easier to remember, in particular
> for the case when an approval needs to be revoked because some change
> was added (or even force-pushed) after approval.
>
>
>
>> Another issue I encountered was with the "closed: *" labels. "closed"
>> doesn't quite seem right to me. Whether something is closed or open is
>> somewhat independent of the states that those labels convey. For example
>> we might want to label something as "not a bug" but leave it open for a
>> little while to allow the reporter to respond or argue why it really
>> should be treated as a bug. Similarly with "wont fix" and maybe even
>> "duplicate".
>
> Actually the 'rejected: *' prefix would be the most appropriate. I just hesitated
> because it sounded so unfriendly. If you have a more friendly proposal, I'd be
> happy to hear about it. Otherwise I would just suggest to use it instead of
> 'closed: *'.
>
>
> Matthias
>



More information about the openssl-project mailing list