Hi Udara,

Thanks for merging !

This should because there are 3 commits in the carbon-store master branch
since we forked the branch.

I will let you know about the stability of this branch after few test
rounds.

Thanks
Rushmin

On Tue, Aug 25, 2015 at 3:38 PM, Udara Rathnayake <[email protected]> wrote:

> Hi Rushmin,
>
> I have merged the PR. While browsing app-manager branch I noticed
> following "This branch is 1270 commits ahead, 3 commits behind master."
> Is this "3 commits behind master" part correct? It might be due to the
> fact that we have preserved git history from both master of carbon-store
> and app-manger of product-es.
>
> Anyway if all features in this branch work for App Manager as expected we
> should not worry about that. Let's remove the app-manager branch from
> product-es once everything looks stable.
>
> Regards,
> UdaraR
>
> On Tue, Aug 25, 2015 at 3:30 PM, Rushmin Fernando <[email protected]>
> wrote:
>
>>
>> Hi Udara and Manu,
>>
>> As per our discussion on $subject, i moved the code and fixed the maven
>> group ids.
>>
>> Please review and merge the pull request [1]
>>
>> Thanks
>> Rushmin
>>
>>
>> [1] - https://github.com/wso2/carbon-store/pull/168
>>
>>
>> --
>> *Rushmin Fernando*
>> *Technical Lead*
>>
>> WSO2 Inc. <http://wso2.com/> - Lean . Enterprise . Middleware
>>
>> email : [email protected]
>> mobile : +94772310855
>>
>>
>>
>


-- 
*Rushmin Fernando*
*Technical Lead*

WSO2 Inc. <http://wso2.com/> - Lean . Enterprise . Middleware

email : [email protected]
mobile : +94772310855
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to