[ https://issues.apache.org/jira/browse/BEAM-9175?focusedWorklogId=378424&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-378424 ]
ASF GitHub Bot logged work on BEAM-9175: ---------------------------------------- Author: ASF GitHub Bot Created on: 28/Jan/20 19:27 Start Date: 28/Jan/20 19:27 Worklog Time Spent: 10m Work Description: Ardagan commented on issue #10684: [BEAM-9175] Introduce an autoformatting tool to Python SDK URL: https://github.com/apache/beam/pull/10684#issuecomment-579414088 > > There is still a number of pylint issues `Wrong continued indentation`. Most of them appear because of how lambda is formatted. For example: > > https://github.com/apache/beam/blob/7db61fbf2dd6eac4ffb542e684260edf0d892fea/sdks/python/apache_beam/io/gcp/bigquery_test.py#L908-L910 > > > > I don't know yet how to deal with it. Unless there is a knob for this, I'll just put a # yapf: disable comments in these places. > > If we adopt yapf's indentation style, then we can disable the same check in pylint. If we utilize autoformatter, especially on pre-commit, it would be feasible to disable all formatting checks on linter. They become obsolete. ---------------------------------------------------------------- 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: 378424) Time Spent: 4h (was: 3h 50m) > 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 > 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)