I'm still working on revisions with the author of the PR.
But I have a question for Nick:
Does this change belong in 0.4.3 or 0.4.4?
I think catching python syntax errors is a good idea.
And this change could enforce python 3 compatibility in all our scripts, if we change the python version to a recent python 3. (But I think we should do that in a separate ticket, in case there are any errors we need to fix.)
I think 0.4.4 might be a better idea, and here's why: new code-checking tooling often comes with significant false positives or similar issues to work around, and often takes a few rounds of revision till it works right everywhere.
With that in mind, I think we should aim to do this in 0.4.4, and only do things in 0.4.3 that stabilize it. But I could probably be persuaded elsewise if I'm missing something here.
I think 0.4.4 is a good release to target. This is an active area right now, and there are a bunch of new volunteers, so let's do it right.
That said, if we could open the merge window for 0.4.4 soon, then it would help these volunteers collaborate from master (rather than working off each other's branches).
Trac: Summary: Travis CI: Use flake8 to find Python syntax errors to Travis CI: Find and fix Python syntax errors Milestone: Tor: 0.4.3.x-final to Tor: 0.4.4.x-final