Re: [GitHub] cloudstack pull request: Squashing two commits in to single commit

2016-02-15 Thread Sanjeev N
LGTM based on PR#667 On Mon, Feb 15, 2016 at 5:37 PM, DaanHoogland wrote: > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/1411#issuecomment-184182132 > > @sanjeevneelarapu If you LGTM this there are 2 LGTM here so please go > ahead

Re: [GitHub] cloudstack pull request: Squashing two commits in to single commit

2016-02-14 Thread Sanjeev N
Is it okay to merge this PR or we need to wait for two LGTMs, since original PR 667 already has two LGTMs ? On Fri, Feb 12, 2016 at 1:59 PM, remibergsma wrote: > Github user remibergsma commented on the pull request: > > https://github.com/apache/cloudstack/pull/1411#issuecomment-183226537 >