You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@qpid.apache.org by GitBox <gi...@apache.org> on 2020/10/27 13:04:42 UTC

[GitHub] [qpid-dispatch] jiridanek commented on pull request #895: DISPATCH-1814 Autofix for flake8 E701 multiple statements on a single line

jiridanek commented on pull request #895:
URL: https://github.com/apache/qpid-dispatch/pull/895#issuecomment-717228696


   I read through the diff and it looks ok. In the future, it might be reasonable to let the tool go through all whitespace changes in one go; autopep8 should be trustworthy enough to simply do that.
   
   There is one problem with merging these changes now; the feature branch for http2 support. Having these formatting changes on master would make it harder to merge that branch later.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org