[ https://issues.apache.org/jira/browse/BEAM-4394?focusedWorklogId=116752&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-116752 ]
ASF GitHub Bot logged work on BEAM-4394: ---------------------------------------- Author: ASF GitHub Bot Created on: 28/Jun/18 04:00 Start Date: 28/Jun/18 04:00 Worklog Time Spent: 10m Work Description: swegner commented on a change in pull request #5797: [BEAM-4394] Spotless Java everywhere URL: https://github.com/apache/beam/pull/5797#discussion_r198701786 ########## File path: examples/java/src/main/java/org/apache/beam/examples/MinimalWordCount.java ########## @@ -104,9 +104,11 @@ public static void main(String[] args) { .apply(Count.perElement()) // Apply a MapElements transform that formats our PCollection of word counts into a // printable string, suitable for writing to an output file. - .apply(MapElements Review comment: It's a bit unfortunate that there's no way to preserve manually formatted line-breaks like this, particularly for examples. But I think it's a small price to pay. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 116752) Time Spent: 50m (was: 40m) > Consider enabling spotless java format throughout codebase > ---------------------------------------------------------- > > Key: BEAM-4394 > URL: https://issues.apache.org/jira/browse/BEAM-4394 > Project: Beam > Issue Type: New Feature > Components: build-system > Reporter: Kenneth Knowles > Priority: Major > Time Spent: 50m > Remaining Estimate: 0h > > "Spotless" can enforce - and automatically restore - automatic Java > formatting. Whenever formatting is off, it tells a user the exact command to > fix it. > It isn't (just) about code layout, it is about automation. We have pretty > strict style rules enforced by checkstyle. The most efficient way to fix up a > file is with autoformat. But if the autoformat hits a bunch of irrelevant > lines, that is annoying for a reviewer and obscures git blame. > If we enforce autoformat all the time, then it makes sure that autoformatting > a particular PR has minimal effects and is always safe to do. -- This message was sent by Atlassian JIRA (v7.6.3#76005)