Prevent unused-import for six.with_metaclass

[jacobtylerwalls]

Type of Changes

	Type
✓	🔨 Refactoring

Description

See pylint-dev/astroid#1622. This change prevents regressing with a false positive unused-import for six.with_metaclass(...) patterns in anticipation of upgrading astroid.

No changelog since there should be no user-facing change (no regression).

Resolves #5935 (actually resolved and tested in pylint-dev/astroid/pull/1622).

[coveralls]

Pull Request Test Coverage Report for Build 2561155549

• 6 of 8 (75.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.01%) to 95.314%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
pylint/checkers/variables.py	6	8	75.0%

Totals
Change from base Build 2560695451:	-0.01%
Covered Lines:	16658
Relevant Lines:	17477

---

💛 - Coveralls

[jacobtylerwalls]

Needs the next minor version of astroid to actually get the line coverage.

[github-actions]

🤖 Effect of this PR on checked open source code: 🤖

Effect on pytest:
The following messages are now emitted:

1. redefined-variable-type:
   Redefinition of importhook type from _distutils_hack.DistutilsMetaFinder to _pytest.assertion.DummyRewriteHook
   https://github.com/pytest-dev/pytest/blob/main/src/_pytest/assertion/__init__.py#L69

Effect on sentry:
The following messages are now emitted:

1. no-member:
   Instance of 'module' has no 'file' member
   https://github.com/getsentry/sentry/blob/master/src/sentry/utils/distutils/commands/base.py#L37

The following messages are no longer emitted:

1. unused-import:
   Unused ActorTuple imported from sentry.models
   https://github.com/getsentry/sentry/blob/master/src/sentry/models/projectownership.py#L9

This comment was generated for commit 0bfa1df

[jacobtylerwalls]

Superseded by #7153