On Fri, Jul 11, 2014 at 1:19 AM, Jesse Meek <jesse.m...@canonical.com>
wrote:

>  I polished off a script this afternoon to check that imports are group
> correctly. It deals with comments, if the groups are in the wrong order, if
> there are too many groups and several other edge cases. I *think* I don't
> have any false positives.
>
> Here is the output: http://pastebin.ubuntu.com/7778556/ (Found 31 errors
> in 26 files)
>
> And the PR fixing them: https://github.com/juju/juju/pull/292
>

Cool. It would be nice to have this in the pre-push hook (or maybe
pre-commit, restricted to modified files only).


> Cheers!
>
> --
> Juju-dev mailing list
> Juju-dev@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>
>
-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev

Reply via email to