apply pylint and add to CI about cbc-casper HOT 13 CLOSED

I'm in favor of changing pylint settings to allow for line lengths up to 100 char instead of default 80. I work on a few python projects (web3.py, pyevm) that do this. Saves me a bit of headache and looks good on most screens.

Thoughts?

from cbc-casper.

I'll be focussing on organization and refactors for the time being. Probably one level deeper than you.

@naterush Will continue pushing the experimental dev -- oracles, simulations, etc.

from cbc-casper.

from cbc-casper.

yes 100%

from cbc-casper.

hey @zramsay, you going to be just doing review or will you be doing some dev too?

blockchain is our defacto branch until issue 26 is done. Assign yourself to an issue if you are going to pick it up.

Thanks for the issues/review :)

from cbc-casper.

haha yeah i discovered the "correct" branch. We should use master/develop once its merged. I'll be doing "light" dev work (e.g. tackling linting/useability/documentation, etc)

from cbc-casper.

Cool! Agreed on master/develop.

from cbc-casper.

Check out flake8. does lint+

from cbc-casper.

@djrtwo I think it's totally reasonable to push the char limit up. The 80 char limit is too restrictive now that everyone has retina displays

from cbc-casper.

it's 100 by default already, at least when I run pylint.

from cbc-casper.

from cbc-casper.

should we also pylint tests ?

from cbc-casper.

Closing. Will open new issue about getting CI working again.

from cbc-casper.