[BEAM-9058] Line-too-long Python lint checks are no longer working.

• Type: Bug
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: Missing
• Component/s: sdk-py-core
• Assignee: Unassigned
• Reporter: Valentyn Tymofieiev
• Created: 06/Jan/20 23:23
• Updated: 13/Jan/22 21:52
• Resolved: 28/Sep/21 23:23

Description

Without getting into a question whether 80 is a reasonable limit, what are the reasons to treat  type annotations differently than other parts of Python code (including comments), which have a 80 character limit?  

 

Places that need to be fixed (PR coming):

```

> Task :sdks:python:test-suites:tox:py37:lintPy37
 
 ************* Module apache_beam.testing.test_stream_it_test
 apache_beam/testing/test_stream_it_test.py:50:0: C0301: Line too long (106/80) (line-too-long)
 ************* Module apache_beam.io.filesystems
 apache_beam/io/filesystems.py:99:0: C0301: Line too long (101/80) (line-too-long)
 apache_beam/io/filesystems.py:100:0: C0301: Line too long (111/80) (line-too-long)
 ************* Module apache_beam.runners.portability.fn_api_runner
 apache_beam/runners/portability/fn_api_runner.py:115:0: C0301: Line too long (114/80) (line-too-long)
 ************* Module apache_beam.transforms.core
 apache_beam/transforms/core.py:1307:0: C0301: Line too long (116/80) (line-too-long)
 apache_beam/transforms/core.py:2271:0: C0301: Line too long (95/80) (line-too-long)
 apache_beam/transforms/core.py:2272:0: C0301: Line too long (90/80) (line-too-long)
 ************* Module setup
 setup.py:141:0: C0301: Line too long (81/80) (line-too-long)

------------------------------------
 

Issue Links

• links to: GitHub Pull Request #10510