Simulation report

[gonzalobenegas]

No description provided.

[fscottfoti]

Matt this merge looks ok to me, but the build is failing. Is that to be expected? Do you want to take a look at this and then go ahead and merge?

[jiffyclub]

Builds are passing at the moment, so if Travis reports a failure something is wrong. In this case it was PEP8 formatting issues. I've fixed them here, but in the future Python code should follow PEP8 guidelines: http://legacy.python.org/dev/peps/pep-0008/.

You can use a command line tool like pep8 to check for formatting issues before before committing. (I have the Travis PEP8 checks configured with a maximum line length of 100 characters, but lines over 79 characters should be rare.)

[gonzalobenegas]

I'll definitely use pep8 next time, thanks.

On 25/03/14 17:09, Matt Davis wrote:

Builds are passing at the moment, so if Travis reports a failure
something is wrong. In this case it was PEP8 formatting issues. I've
fixed them here, but in the future Python code should follow PEP8
guidelines: http://legacy.python.org/dev/peps/pep-0008/.

You can use a command line tool like pep8
http://pep8.readthedocs.org/en/latest/ to check for formatting
issues before before committing. (I have the Travis PEP8 checks
configured with a maximum line length of 100 characters, but lines
over 79 characters should be rare.)

—
Reply to this email directly or view it on GitHub
#13 (comment).