-0
I think there is no harm in keeping the PRs open which are not merged yet, as a
reference which can be picked up by someone over time and merged/closed later.
For example, here is an interesting PR that is more than a month old but if you
close it we risk forgetting about it:
Hi all,
This PR includes the ones we can close now:
https://github.com/apache/cloudstack/pull/706
Any LGTM’s so we can merge it?
Thanks,
Remi
On 17 Aug 2015, at 18:56, Rajani Karuturi
raj...@apache.orgmailto:raj...@apache.org wrote:
+1 for auto closing.
I also agree with Boris that we need
Hi all,
There are several PRs that are quite old. They haven't been updated by their
author for over a month and there was no response to comments made.
As a RM, I want to maintain an as-short-as-possible list of PRs that is
actively worked on. It is perfectly fine if a PR is open for a longer
+1 for auto closing.
I also agree with Boris that we need to distinguish discarded vs. Merged
prs.
On Mon, Aug 17, 2015 at 21:51 PM, Mike Tutkowski
mike.tutkow...@solidfire.com wrote:
+1 Sounds reasonable
On Mon, Aug 17, 2015 at 8:25 AM, Remi Bergsma rberg...@schubergphilis.com
javascript:;
somesh.na...@citrix.com
wrote:
+1
Regards,
Somesh
-Original Message-
From: Remi Bergsma [mailto:rberg...@schubergphilis.com]
Sent: Monday, August 17, 2015 10:26 AM
To: dev@cloudstack.apache.org
Subject: [PROPOSAL] Closing PRs older than 1
+1
Regards,
Somesh
-Original Message-
From: Remi Bergsma [mailto:rberg...@schubergphilis.com]
Sent: Monday, August 17, 2015 10:26 AM
To: dev@cloudstack.apache.org
Subject: [PROPOSAL] Closing PRs older than 1 month and without activity
Hi all,
There are several PRs that are quite old
+1 Sounds reasonable
On Mon, Aug 17, 2015 at 8:25 AM, Remi Bergsma rberg...@schubergphilis.com
wrote:
Hi all,
There are several PRs that are quite old. They haven't been updated by
their author for over a month and there was no response to comments made.
As a RM, I want to maintain an
] Closing PRs older than 1 month and without activity
Hi all,
There are several PRs that are quite old. They haven't been updated by
their author for over a month and there was no response to comments made.
As a RM, I want to maintain an as-short-as-possible list of PRs
Hi Rajani,
If we truly “merge” PRs, which I think we should do (instead of applying a
patch) then those will be in state “Merged” (purple) versus “Closed” (red).
Regards,
Remi
On 17 Aug 2015, at 18:56, Rajani Karuturi
raj...@apache.orgmailto:raj...@apache.org wrote:
+1 for auto closing.
I