On Fri, Oct 17, 2014 at 6:40 AM, Andrey Kurilin <[email protected]> wrote:
> Hi everyone! > > I'm working on enabling E12* PEP8 rules in novaclient(status of my work > listed below). Imo, PEP8 rules should be ignored only in extreme cases/for > important reasons and we should decrease a number of ignored rules. This > helps to keep code in more strict, readable form, which is very important > when working in community. > > While working on rule E126, we started discussion with Joe Gordon about > demand of these rules. I have no idea about reasons of why they should be > ignored, so I want to know: > - Why these rules should be ignored? > - What do you think about enabling these rules? > I found the source of my confusion. See my inline comments in https://review.openstack.org/#/c/122888/10/tox.ini Hopefully this patch should clarify things: https://review.openstack.org/129677 > > Please, leave your opinion about E12* rules. > > Already enabled rules: > E121,E125 - https://review.openstack.org/#/c/122888/ > E122 - https://review.openstack.org/#/c/123830/ > E123 - https://review.openstack.org/#/c/123831/ > > Abandoned rule: > E124 - https://review.openstack.org/#/c/123832/ > > Pending review: > E126 - https://review.openstack.org/#/c/123850/ > E127 - https://review.openstack.org/#/c/123851/ > E128 - https://review.openstack.org/#/c/127559/ > E129 - https://review.openstack.org/#/c/123852/ > > > -- > Best regards, > Andrey Kurilin. > > _______________________________________________ > OpenStack-dev mailing list > [email protected] > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > >
_______________________________________________ OpenStack-dev mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
