tengqm commented on PR #5837: URL: https://github.com/apache/gravitino/pull/5837#issuecomment-2540210062
> IDE wrap lines, so that's not really an issue. As I explained elsewhere, if you break up lines, tools like grammar and style checkers that we use will not work. Some people who contribute to this project don't have English as their first language and rely heavily on these tools. No. It is an issue, and a very annoying issue. Please refer [this comment](https://github.com/apache/gravitino/pull/5840#discussion_r1881158655). Different people (including developers) see different things (I'm not using your IDE), that is an additional issue. This is like the TAB usage scenario. Grammar or style checker is not a valid reason for this change. I was not encouraging breaking a long line in the middle. I was suggesting to break long lines APPROPRIATELY, at statement boundary, at punctuation. -- 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: commits-unsubscr...@gravitino.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
