Skip to content

Commit e62213a

Browse files
committed
Remove stray pylint config in pyproject.toml
In the recently merged Qiskit#15603 we migrated to using ruff as are sole Python linting tool in Qiskit. However that PR neglected to remove the stray pylint configuration in the pyproject.toml. This commit fixes that oversight and removes this configuration entries from the file.
1 parent e81a3b3 commit e62213a

1 file changed

Lines changed: 0 additions & 56 deletions

File tree

pyproject.toml

Lines changed: 0 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -484,59 +484,6 @@ ignore = [
484484
# Visualization functions often subprocess to visualization tools
485485
"qiskit/visualization/*" = ["S603", "S607"]
486486

487-
488-
[tool.pylint.main]
489-
extension-pkg-allow-list = [
490-
"numpy",
491-
"qiskit._accelerate",
492-
"qiskit._qasm2",
493-
"qiskit._qasm3",
494-
# We can't allow pylint to load qiskit._qasm2 because it's not able to
495-
# statically resolve the cyclical load of the exception and it bugs out.
496-
"retworkx",
497-
"rustworkx",
498-
]
499-
load-plugins = ["pylint.extensions.docparams", "pylint.extensions.docstyle", "pylint.extensions.bad_builtin"]
500-
py-version = "3.10" # update it when bumping minimum supported python version
501-
502-
[tool.pylint.basic]
503-
good-names = ["a", "b", "i", "j", "k", "d", "n", "m", "ex", "v", "w", "x", "y", "z", "Run", "_", "logger", "q", "c", "r", "qr", "cr", "qc", "nd", "pi", "op", "b", "ar", "br", "p", "cp", "ax", "dt", "__unittest", "iSwapGate", "mu"]
504-
method-rgx = "(([a-z_][a-z0-9_]{2,49})|(assert[A-Z][a-zA-Z0-9]{2,43})|(test_[_a-zA-Z0-9]{2,}))$"
505-
variable-rgx = "[a-z_][a-z0-9_]{1,30}$"
506-
507-
[tool.pylint.format]
508-
max-line-length = 105 # default 100
509-
510-
[tool.pylint."messages control"]
511-
disable = [
512-
# intentionally disabled:
513-
"spelling", # too noisy
514-
"fixme", # disabled as TODOs would show up as warnings
515-
"protected-access", # disabled as we don't follow the public vs private convention strictly
516-
"duplicate-code", # disabled as it is too verbose
517-
"redundant-returns-doc", # for @abstractmethod, it cannot interpret "pass"
518-
"too-many-instance-attributes", "too-many-arguments", "too-few-public-methods", "too-many-ancestors",
519-
"unnecessary-pass", # allow for methods with just "pass", for clarity
520-
"unnecessary-dunder-call", # do not want to implement
521-
"no-else-return", # relax "elif" after a clause with a return
522-
"docstring-first-line-empty", # relax docstring style
523-
"import-outside-toplevel", "import-error", # overzealous with our optionals/dynamic packages
524-
"consider-using-max-builtin", "consider-using-min-builtin", # unnecessary stylistic opinion
525-
# TODO(#9614): these were added in modern Pylint. Decide if we want to enable them. If so,
526-
# remove from here and fix the issues. Else, move it above this section and add a comment
527-
# with the rationale
528-
"no-member", # for dynamically created members
529-
"not-context-manager",
530-
"unnecessary-lambda-assignment", # do not want to implement
531-
]
532-
533-
enable = [
534-
"use-symbolic-message-instead"
535-
]
536-
537-
[tool.pylint.spelling]
538-
spelling-private-dict-file = ".local-spellings"
539-
540487
[tool.coverage.report]
541488
exclude_also = [
542489
"def __repr__", # Printable epresentational string does not typically execute during testing
@@ -545,6 +492,3 @@ exclude_also = [
545492
"if TYPE_CHECKING:", # Code that only runs during type checks
546493
"@abstractmethod", # Abstract methods are not testable
547494
]
548-
549-
[tool.pylint.deprecated_builtins]
550-
bad-functions = ["print"]

0 commit comments

Comments
 (0)