[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-26 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-134964553 @bhaisaab @remibergsma : Rohit has some good points there and some that can be avoided. Noticably a PR that hasn't gotten any attention from the community shoul

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-26 Thread remibergsma
Github user remibergsma closed the pull request at: https://github.com/apache/cloudstack/pull/706 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-26 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-134919031 @bhaisaab OK, let's leave it for now and discuss later. I will close the PR. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-26 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-134915109 All, let's not do this for multiple reasons; - by closing a PR which has not got attention we risk forgetting about it and losing the contribution - gen

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-17 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-131868071 @DaanHoogland Weekly basis is OK with me! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-131866061 I did not send a :-1: to dev@. I did not omit that mail by mistake (sorry for the double negative). the proposal seems radical (17 july is phased out on 17 Aug

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-17 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-131864636 @DaanHoogland That's true. Although @wilderrodrigues will take over the PR and needs to create a new one. But let's stick to the proposal indeed. I'll remove #50

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-17 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/706#issuecomment-131860676 https://github.com/apache/cloudstack/pull/508 was worked on 14 days ago by @resmo and @wilderrodrigues --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Closing PRs older than 1 month and withou...

2015-08-17 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/706 Closing PRs older than 1 month and without activity As proposed here: http://cloudstack.markmail.org/search/?q=#query:+page:1+mid:rkptskqvajs7n7zj+state:results This will close thes