I would also advise pinning the version of mccabe we’re using. Mccabe was originally a proof-of-concept script that Ned Batchelder wrote and which Tarek Ziade vendored into Flake8. After we split it out in v2 of Flake8, we’ve found several (somewhat serious) reporting problems with the tool. Currently the package owner on PyPI hasn’t granted me permissions to release a new version of the package, but we have several fixes in the repository: https://github.com/flintwork/mccabe. The changes are somewhat drastic but they should reduce the average function/method’s complexity by 1 or 2 points. I’m going to bother Florent again to give me permission to release the package since it has been far too long since a release has been cut.
For what it’s worth, Florent doesn’t pay close attention to GitHub notifications so chiming in (or creating) issues on mccabe to release a new version will only spam *me*. So please don’t pile on to anything existing or create a new one. Cheers, Ian On 10/17/14, 12:39 AM, "Michael Davies" <mich...@the-davies.net> wrote: >On Fri, Oct 17, 2014 at 2:39 PM, Joe Gordon ><joe.gord...@gmail.com> wrote: > > > >First step in fixing this, put a cap on it: ><http://goog_106984861>https://review.openstack.org/129125 > > > > > > > >Thanks Joe - I've just put up a similar patch for Ironic: >https://review.openstack.org/129132 <https://review.openstack.org/129132> > > > >-- >Michael Davies mich...@the-davies.net >Rackspace Australia > _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
