xkrogen commented on pull request #35379: URL: https://github.com/apache/spark/pull/35379#issuecomment-1029211170
> Sorry for that. I run ./dev/scalafmt hoping to fix the linting errors. (I was surprised that it didn't fix all linting errors.) Is it possible that we make formatting an automatic step in compiling and make sure the linting rules are consistent with the formatting rules? No problem ... Totally agreed that the formatter and linting rules should be in sync. I'm the one who wrote most of the code that had the whitespace updated here and I usually assume that the linter will catch any issues :) I can put up a PR addressing the formatting issues. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
