cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sebastien Goasguen <run...@gmail.com>
Subject Re: easy bug to fix for new comer
Date Thu, 20 Jun 2013 15:44:46 GMT
thanks, I will check your patches now


On Jun 20, 2013, at 10:54 AM, Daan Hoogland <DHoogland@schubergphilis.com> wrote:

> Halting now, will take on the bigger ones tonight/-morow
> 
> -----Original Message-----
> From: Daan Hoogland [mailto:DHoogland@schubergphilis.com] 
> Sent: donderdag 20 juni 2013 16:12
> To: 'dev@cloudstack.apache.org'
> Subject: RE: easy bug to fix for new comer
> 
> I noticed some are very small. Doing those first and I am still doing them one file at
a time!
> 
> -----Original Message-----
> From: Prasanna Santhanam [mailto:tsp@apache.org] 
> Sent: donderdag 20 juni 2013 16:02
> To: dev@cloudstack.apache.org
> Subject: Re: easy bug to fix for new comer
> 
> cloudstackConnection was pep8 compliant but seems to have flouted it again. I'll look
at setting up a jenkins job for style check when the full pep8 compliance is done.
> 
> On Thu, Jun 20, 2013 at 01:42:36PM +0000, Daan Hoogland wrote:
>> Sure,
>> 
>> One file at a time!
>> 
>> -----Original Message-----
>> From: Sebastien Goasguen [mailto:runseb@gmail.com]
>> Sent: donderdag 20 juni 2013 14:35
>> To: dev@cloudstack.apache.org
>> Subject: easy bug to fix for new comer
>> 
>> Hi,
>> 
>> Here is an easy bug to fix for a newcomer to cloudstack:
>> 
>> https://issues.apache.org/jira/browse/CLOUDSTACK-3096
>> 
>> install pep8 on your machine
>> git clone cloudstack repo
>> go to the marvin directory
>> 
>> run pep8 like I show in the bug.
>> 
>> edit the python scripts to fix the errors
>> 
>> once pep8 is clean, git commit everything, create a patch and send to review board.
>> 
>> Any takers ?
>> 
>> -Sebastien
> 
> --
> Prasanna.,
> 
> ------------------------
> Powered by BigRock.com
> 


Mime
View raw message