Re: capitalized method name

2015-07-14 Thread Daan Hoogland
thanks, i made https://github.com/apache/cloudstack/pull/584 On Tue, Jul 14, 2015 at 5:45 AM, Jayapal Reddy Uradi wrote: > Hi Daan, > > Please go ahead and change the name. > Thanks for notifying. > > -Jayapal > > > On 13-Jul-2015, at 8:45 PM, Daan Hoogland wrote: > >> Jayapal, >> >> I didn't no

Re: capitalized method name

2015-07-13 Thread Jayapal Reddy Uradi
Hi Daan, Please go ahead and change the name. Thanks for notifying. -Jayapal On 13-Jul-2015, at 8:45 PM, Daan Hoogland wrote: > Jayapal, > > I didn't notice this before but in PR 568, in commit > 638885a9de0fd651351f0b16cbdd9fc30c4a187d, you added a method name > starting with capital 'A' to

capitalized method name

2015-07-13 Thread Daan Hoogland
Jayapal, I didn't notice this before but in PR 568, in commit 638885a9de0fd651351f0b16cbdd9fc30c4a187d, you added a method name starting with capital 'A' to citrixrecource base. Is this on purpose and can we change it to adhere to convention? public boolean AttachConfigDriveToMigratedVm(Conne