@tgamblin - I am currently going through cmd/create.py to fix the template to follow your new rules. On checking that very file now for coding rules, yet another whitespace checker (E 241) really kills the nice formatting. Same with the line length, where I wouldn't like splitting up some of the strings in there into two lines. I even have a syntax error false-positive. Just have a look at your logs ;-)

Seems the configure commands have very late binding. My attempt to fix doesn't help -> closing

Re-opening after discussion on #924 . Requires converging on rules before finishing

@hegner: I like the narrow column width because it gets me lots of terminals side-by-side on a laptop and even more on a big screen. It's also in PEP8. I just added E241 to the exception list in #929, and i added a share/spack/qa/run-flake8 script that will run the checks locally so it's easier to iterate.

@tgamblin - thanks. makes it much easier.

@hegner: We've added an exception for import * in the PEP8 rules. The intent with the packages is really to make it easy for packagers -- I think allowing them to always use from spack import * is fine. So closing this one -- unless you really want explicit imports in packages for some non-PEP8 reason :)

@tgamblin - thanks. happy you don't want to be that strict.