+1

I've also created a script that trawls our PRs to both label and create a 
report.  Right now it uses the Draft 'type' to figure out what's 'coming soon'.
Ps. Once polished I plan to contribute the docker containers that takes away 
all 'environment creation' hassles.


paul.an...@shapeblue.comĀ 
www.shapeblue.com
3 London Bridge Street,  3rd floor, News Building, London  SE1 9SGUK
@shapeblue
  
 


-----Original Message-----
From: Abhishek Kumar <abhishek.ku...@shapeblue.com> 
Sent: 17 February 2020 11:09
To: Daan Hoogland <daan.hoogl...@gmail.com>; dev <dev@cloudstack.apache.org>
Subject: Re: [DISCUSS]/[PROPOSAL] draft PRs

+1
________________________________
From: Daan Hoogland <daan.hoogl...@gmail.com>
Sent: 14 February 2020 17:33
To: dev <dev@cloudstack.apache.org>
Subject: [DISCUSS]/[PROPOSAL] draft PRs

devs, I thought I had already sent a mail about this but i cannot find it.
I'm sure i had mentioned it somewhere (probably on github).
this is a follow up on [1] and hopefully you'll agree, a slight improvement.

here it comes:

At the moment we are creating PRs with a [WIP] or [DO NOT MERGE] tag in the 
title. This title stands the chance of being merged once we agree the PR is 
ready for merge. It also clutters the title.

Github has introduced a nice feature a while ago; draft PR. When creating a PR 
you can opt not to open it for merge but as draft. Choose a button left of the 
"Create pull request" button, marked "Create draft PR". It will be a full PR 
with all CI and discussion possibilities open. The only difference is the merge 
button being disabled. One will than have to make/mark it "ready for merge" 
before it *can* be merged.

[1]
https://lists.apache.org/thread.html/f3f0988907f85bfc2cfcb0fbcde831037f9b1cb017e94bcdddd68932%40%3Cdev.cloudstack.apache.org%3E
please shoot any comments you may have back at me, thanks

--
Daan

abhishek.ku...@shapeblue.com
www.shapeblue.com
3 London Bridge Street,  3rd floor, News Building, London  SE1 9SGUK @shapeblue
  
 

Reply via email to