Update pylint to 2.8.0

[cdce8p]

Proposed change

Update pylint and astroid to their latest versions.
Changelog: http://pylint.pycqa.org/en/latest/whatsnew/changelog.html#what-s-new-in-pylint-2-8-0
Full diff: pylint-dev/pylint@pylint-2.7.4...pylint-2.8.0

New features

• New checkers: consider-using-min-builtin, consider-using-max-builtin, consider-using-with
• Improved checker for deprecated function arguments. E.g., the loop argument for a lot of asyncio functions
• New optional extension: typing, with the checkers deprecated-typing-alias, consider-using-alias, consider-alternative-union-syntax. These allow us to easily check for deprecated typing aliases (PEP 585) and old union syntax (PEP 604). Most of the resulting changes have already been merged with Update typing syntax #49480

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New integration (thank you!)
• New feature (which adds functionality to an existing integration)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml

Additional information

• This PR fixes or closes issue: fixes #
• This PR is related to issue:
• Link to documentation pull request:

Checklist

• The code change is tested and works locally.
• Local tests pass. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.
• I have followed the development checklist
• The code has been formatted using Black (black --fast homeassistant tests)
• Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• The manifest file has all fields filled out correctly.
  Updated and included derived files by running: python3 -m script.hassfest.
• New or updated dependencies have been added to requirements_all.txt.
  Updated by running python3 -m script.gen_requirements_all.
• Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

• No score or internal
• 🥈 Silver
• 🥇 Gold
• 🏆 Platinum

To help with the load of incoming pull requests:

• I have reviewed two other open pull requests in this repository.

[MartinHjelmare]

Looks good!

[bdraco]

Thanks for doing the upgrade 👍

Looks like change to with for subprocesses is causing a problem with the timeouts in the test

tests/components/command_line/test_notify.py is failing locally with this change

[cdce8p]

Thanks for doing the upgrade 👍

Looks like change to with for subprocesses is causing a problem with the timeouts in the test

tests/components/command_line/test_notify.py is failing locally with this change

Already working on it 😄

[cdce8p]

Looks like change to with for subprocesses is causing a problem with the timeouts in the test
tests/components/command_line/test_notify.py is failing locally with this change

Already working on it 😄

Think I got it now. @bdraco Would you mind taking a quick look at the last two commits?

[bdraco]

Looks good 👍
Thanks!