[ https://issues.apache.org/jira/browse/BEAM-9175?focusedWorklogId=378727&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-378727 ]
ASF GitHub Bot logged work on BEAM-9175: ---------------------------------------- Author: ASF GitHub Bot Created on: 29/Jan/20 11:05 Start Date: 29/Jan/20 11:05 Worklog Time Spent: 10m Work Description: kamilwu commented on issue #10684: [BEAM-9175] Introduce an autoformatting tool to Python SDK URL: https://github.com/apache/beam/pull/10684#issuecomment-579705404 > Is it possible to split change to one that adds formatter and one that shows what formatting actually looks like? It's already split like this. The first commit only adds the formatter, and the second commit is the one that applies formatting changes to the code. Or do you mean creating a separate PR? ---------------------------------------------------------------- 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: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 378727) Time Spent: 4h 10m (was: 4h) > Introduce an autoformatting tool to Python SDK > ---------------------------------------------- > > Key: BEAM-9175 > URL: https://issues.apache.org/jira/browse/BEAM-9175 > Project: Beam > Issue Type: Improvement > Components: sdk-py-core, sdk-py-harness > Reporter: Michał Walenia > Assignee: Kamil Wasilewski > Priority: Major > Time Spent: 4h 10m > Remaining Estimate: 0h > > It seems there are three main options: > * black - very simple, but not configurable at all (except for line length), > would drastically change code style > * yapf - more options to tweak, can omit parts of code > * autopep8 - more similar to spotless - only touches code that breaks > formatting guidelines, can use pycodestyle and flake8 as configuration > The rigidity of Black makes it unusable for Beam. -- This message was sent by Atlassian Jira (v8.3.4#803005)