Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix lint in pylintrc #5952

Merged
merged 2 commits into from
Nov 22, 2022
Merged

Conversation

pavoljuhas
Copy link
Collaborator

  • add missing comma after f-string-without-interpolation
  • remove obsolete checks mixed-indentation and relative-import
    specific to Python 2. Resolve pylint 2.15 complaints about these.

Remove obsolete checks `mixed-indentation` and `relative-import`
specific to Python 2.  Resolve pylint 2.15 complaints about these.
@pavoljuhas pavoljuhas requested review from a team, vtomole and cduck as code owners November 22, 2022 01:48
@CirqBot CirqBot added the Size: XS <10 lines changed label Nov 22, 2022
@CirqBot CirqBot added the size: S 10< lines changed <50 label Nov 22, 2022
@tanujkhattar tanujkhattar merged commit 8ea41a8 into quantumlib:master Nov 22, 2022
@pavoljuhas pavoljuhas deleted the fix-lint-in-pylintrc branch November 22, 2022 20:53
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
* Add missing comma after f-string-without-interpolation

Remove obsolete checks `mixed-indentation` and `relative-import`
specific to Python 2.  Resolve pylint 2.15 complaints about these.

* Clean redundant f-strings that show for fixed pylintrc
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
* Add missing comma after f-string-without-interpolation

Remove obsolete checks `mixed-indentation` and `relative-import`
specific to Python 2.  Resolve pylint 2.15 complaints about these.

* Clean redundant f-strings that show for fixed pylintrc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: S 10< lines changed <50 Size: XS <10 lines changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants