[BEAM-11516] Upgrade to pylint >=2.6.0

• Type: Task
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 2.34.0
• Component/s: sdk-py-core, testing
• Assignee: Benjamin Gonzalez
• Reporter: Brian Hulette
• Created: 23/Dec/20 00:06
• Updated: 04/Oct/21 16:17
• Resolved: 04/Oct/21 15:27

Description

I accidentally ran the latest pylint (2.6.0 as opposed to 2.4.3 which we currently use), and found that it raised a lot of warnings, many of which looked useful. For example:

apache_beam/pvalue.py:500:11: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)
apache_beam/utils/timestamp.py:132:6: W0707: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)

We should address these warnings and run 2.6.0 (or whatever is the latest at time of implementation) continuously

Issue Links

• split to: BEAM-12992 Fix pylint warning: Using open without explicitly specifying an encoding (unspecified-encoding)
• split to: BEAM-12991 Fix pylint warning: Consider explicitly re-raising using the 'from' keyword (raise-missing-from)
• links to: GitHub Pull Request #15612