Re: [DISCUSS] PR targeting for enhancement, feature, bugfix and breaking changes


Florian Hockmann <f...@...>
 

I'm fine with adding labels for features and breaking changes, but where I'm not sure about is this part:

"enhancement" and "bugfix" can go to any branch

Who decides here which branch to target? The contributor? Any reviewer?

I think we should have a policy that specifies which kind of contribution should target which branch and it should ideally be easy to understand.

Am Dienstag, 2. Juli 2019 15:38:29 UTC+2 schrieb Jason Plurad:
Ah, fair enough, enhancement makes sense when explained more fully by example.

On Monday, July 1, 2019 at 9:19:00 AM UTC-4, Oleksandr Porunov wrote:
If an issue suggests a performance improvement, can we label it as a "bugfix"?

On Monday, July 1, 2019 at 3:59:52 PM UTC+3, Jason Plurad wrote:
I think "enhancement" can be easily confused with "feature". I'd suggest to stick with "bugfix" for items that would go into any branch.

On Monday, July 1, 2019 at 7:34:59 AM UTC-4, Oleksandr Porunov wrote:
Hello,

Currently both new features and enhancements are labeled as "enhancement".
I am proposing to introduce new labels: 
- "feature" - a new functionality
- "breaking changes" - breaking changes

After that I am proposing to use the next PR targeting scheme:
- "breaking changes" and "feature" can go only to master branch
- "enhancement" and "bugfix" can go to any branch

I don't think that we need a special voting for this that is why I will wait for 1 week for other comments from contributors. If everyone is OK with it then we can stick to a new PR targeting scheme

Best regards,
Oleksandr

Join janusgraph-dev@lists.lfaidata.foundation to automatically receive all group messages.