Bug 1138481 - Add .pep8.rc with preferred settings to the repository #175
Conversation
|
@whimboo r? |
|
Hey @IamAdiSri! Thank you for uploading the patch as PR. Sorry that I haven't had the time to check it earlier today. But I think I can wait with a deeper review given that the Travis CI test failed. This is something you will have to fix first. But let me have a look at it now. |
| max-line-length = 99 | ||
| show-source = enable | ||
| statistics = enable | ||
| count = enable |
There was a problem hiding this comment.
As I have mentioned already on the bug in Bugzilla we don't want those three boolean keys included. Those are not necessary and seem to also cause the failure with Travis.
Bug 1138481 Created .pep8.rc file Edited .travis.yml file
|
@whimboo r? |
|
That looks great now. Thank you for the updates @IamAdiSri. I will get the commits squashed and merged immediately. |
|
Merged as 9e20028. |
Bug 1138481
Created .pep8.rc file
Edited .travis.yml file