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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
