[pre-commit.ci] pre-commit autoupdate

[pre-commit-ci]

updates:

• github.com/asottile/pyupgrade: v2.37.2 → v2.37.3
• https://github.com/Pierre-Sassoulas/black-disable-checker/: v1.1.0 → v1.1.1
• github.com/PyCQA/flake8: 4.0.1 → 5.0.3

[coveralls]

Pull Request Test Coverage Report for Build 2817777003

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 92.257%

---

Totals
Change from base Build 2776943930:	0.0%
Covered Lines:	9699
Relevant Lines:	10513

---

💛 - Coveralls

[Pierre-Sassoulas]

Regarding the last check disabled I'd like your review @DanielNoord. It was raising:

astroid/nodes/scoped_nodes/scoped_nodes.py:2876:13: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/nodes/node_classes.py:94:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:267:5: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:361:5: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:409:13: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:481:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:607:5: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/inference.py:1150:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/protocols.py:297:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/protocols.py:452:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/protocols.py:530:5: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.
astroid/protocols.py:610:9: B901 Using `yield` together with `return x`. Use native `async def` coroutines or put a `# noqa` comment on this line if this was intentional.

See PyCQA/flake8-bugbear#2 and I think every current warning is a false positive because it's always going to be intentional for legacy code.

[DanielNoord]

What are these three messages?

And why does this bump suddenly raise B901? That PR you mention was merged in 2016?

[Pierre-Sassoulas]

I guess it's a breaking change from flake8 > 5 that change the default vs disabled message. The two other messages are line too long and line break before binary operator (I did not remember but it's in the commit message).

[Pierre-Sassoulas]

Thank you @DanielNoord !