Skip to content

Inconsistency in check/pylint #3716

Closed
Closed
@95-martin-orion

Description

Description of the issue

PR #3698 hit the following pylint errors:

************* Module cirq.study.sweepable
cirq/study/sweepable.py:17:0: W0611: Unused Dict imported from typing (unused-import)
************* Module cirq.optimizers.align_right
cirq/optimizers/align_right.py:17:0: W0611: Unused ops imported from cirq (unused-import)
************* Module cirq.optimizers.align_left
cirq/optimizers/align_left.py:17:0: W0611: Unused ops imported from cirq (unused-import)

These files were not modified in this PR; as far as I can tell, the issues above were introduced in #3643 and #3644. Strangely, those PRs did not get flagged by the pylint checker.

I have confirmed that all three PRs listed here used pylint-2.6.0 in their checks.

How to reproduce the issue

Unclear. It seems like check/pylint behaves inconsistently in PR tests; we should double-check that it's set up correctly.

Cirq version
v0.10.0-dev

Metadata

Assignees

No one assigned

    Labels

    area/ciarea/pylintkind/healthFor CI/testing/release process/refactoring/technical debt items

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions