ibzib commented on pull request #13815: URL: https://github.com/apache/beam/pull/13815#issuecomment-767944135
> Given we have a whitepace lint that doesn't actually fix the problem, I'm inclined to believe we don't have one such thing at present. As far as I know, you're correct that there's no formatter right now. I agree that having a whitespace linter without an autoformatter is less than helpful since it adds to the manual maintenance burden without providing much benefit. > Your preference is noted but I strongly disagree with the sysyphean effort proposed by manually maintaining 80 char flows without automation. To be clear, I'm proposing we don't set any standard until we can both enforce it with a precommit check and fix violations with an automatic formatter. This PR is fine, but I think we can agree that manual maintenance is unsustainable. ---------------------------------------------------------------- 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]
