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

Upgraded to pylint 2.17.5 #266

Merged
merged 1 commit into from
Sep 15, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion icontract/_recompute.py
Original file line number Diff line number Diff line change
Expand Up @@ -1096,7 +1096,7 @@ def visit_Lambda(self, node: ast.Lambda) -> Callable[..., Any]:
"Please make a feature request on https://github.com/Parquery/icontract"
)

def visit_Return(self, node: ast.Return) -> Any: # pylint: disable=no-self-use
def visit_Return(self, node: ast.Return) -> Any:
"""Raise an exception that this node is unexpected."""
raise AssertionError(
"Unexpected return node during the re-computation: {}".format(
Expand Down
2 changes: 1 addition & 1 deletion pylint.rc
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ generated-members=bottle\.request\.forms\.decode,bottle\.request\.query\.decode
max-line-length=120

[MESSAGES CONTROL]
disable=too-few-public-methods,abstract-class-little-used,len-as-condition,bad-continuation,bad-whitespace,duplicate-code,no-else-raise,too-many-locals,too-many-branches,too-many-lines,too-many-arguments,too-many-statements,too-many-nested-blocks,too-many-function-args,too-many-instance-attributes,too-many-public-methods,protected-access,consider-using-in,no-member,consider-using-f-string,use-dict-literal,redundant-keyword-arg,no-self-use
disable=too-few-public-methods,len-as-condition,duplicate-code,no-else-raise,too-many-locals,too-many-branches,too-many-lines,too-many-arguments,too-many-statements,too-many-nested-blocks,too-many-function-args,too-many-instance-attributes,too-many-public-methods,protected-access,consider-using-in,no-member,consider-using-f-string,use-dict-literal,redundant-keyword-arg

2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
install_requires=install_requires,
extras_require={
"dev": [
"pylint==2.13.9",
"pylint==2.17.5",
"tox>=3.0.0",
"pydocstyle>=6.1.1,<7",
"coverage>=4.5.1,<5",
Expand Down
1 change: 0 additions & 1 deletion tests/test_args_and_kwargs_in_contract.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=unnecessary-lambda
Expand Down
1 change: 0 additions & 1 deletion tests/test_error.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=no-self-use
# pylint: disable=unused-variable

import unittest
Expand Down
2 changes: 1 addition & 1 deletion tests/test_for_integrators.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
"""Test logic that can be potentially used by the integrators such as third-party libraries."""

# pylint: disable=no-self-use,missing-docstring
# pylint: disable=missing-docstring
# pylint: disable=invalid-name,unnecessary-lambda

import ast
Expand Down
2 changes: 1 addition & 1 deletion tests/test_inheritance_invariant.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ class A(icontract.DBC):
def __repr__(self) -> str:
return "instance of A"

def some_func(self) -> int: # pylint: disable=no-self-use
def some_func(self) -> int:
return 1

class B(A):
Expand Down
1 change: 0 additions & 1 deletion tests/test_inheritance_postcondition.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=no-self-use

import abc
import sys
Expand Down
1 change: 0 additions & 1 deletion tests/test_inheritance_precondition.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=no-self-use

import abc
import sys
Expand Down
2 changes: 1 addition & 1 deletion tests/test_invariant.py
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ def some_method(self) -> None:
inst.some_method()
self.assertEqual(10, inst.x)

def test_inv_with_empty_arguments(self) -> None: # pylint: disable=no-self-use
def test_inv_with_empty_arguments(self) -> None:
z = 42

@icontract.invariant(lambda: z == 42)
Expand Down
1 change: 0 additions & 1 deletion tests/test_postcondition.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=no-self-use
# pylint: disable=unnecessary-lambda

import abc
Expand Down
1 change: 0 additions & 1 deletion tests/test_precondition.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=no-self-use
# pylint: disable=unnecessary-lambda

import functools
Expand Down
2 changes: 1 addition & 1 deletion tests/test_recompute.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# pylint: disable=missing-docstring,invalid-name,no-self-use
# pylint: disable=missing-docstring,invalid-name
# pylint: disable=unused-argument
import ast
import re
Expand Down
1 change: 0 additions & 1 deletion tests/test_recursion.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=unnecessary-lambda
import unittest
from typing import List
Expand Down
2 changes: 1 addition & 1 deletion tests/test_represent.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# pylint: disable=missing-docstring,invalid-name,no-self-use
# pylint: disable=missing-docstring,invalid-name
# pylint: disable=unused-argument
# pylint: disable=unnecessary-lambda

Expand Down
2 changes: 1 addition & 1 deletion tests/test_snapshot.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# pylint: disable=invalid-name
# pylint: disable=unnecessary-lambda
# pylint: disable=unused-argument
# pylint: disable=no-self-use

import textwrap
import unittest
from typing import List, Optional # pylint: disable=unused-import
Expand Down
2 changes: 1 addition & 1 deletion tests/test_threading.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# pylint: disable=missing-docstring,unnecessary-lambda,no-self-use
# pylint: disable=missing-docstring,unnecessary-lambda
import threading
import unittest

Expand Down
1 change: 0 additions & 1 deletion tests/test_typeguard.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=invalid-name
# pylint: disable=unused-argument

Expand Down
2 changes: 1 addition & 1 deletion tests_3_6/test_represent.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# pylint: disable=missing-docstring,invalid-name,no-self-use
# pylint: disable=missing-docstring,invalid-name
# pylint: disable=unused-argument

import textwrap
Expand Down
1 change: 0 additions & 1 deletion tests_3_8/async/separately_test_concurrent.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=unnecessary-lambda
# pylint: disable=disallowed-name
import asyncio
Expand Down
1 change: 0 additions & 1 deletion tests_3_8/async/test_args_and_kwargs_in_contract.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=unnecessary-lambda
Expand Down
1 change: 0 additions & 1 deletion tests_3_8/async/test_recursion.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
# pylint: disable=missing-docstring
# pylint: disable=no-self-use
# pylint: disable=unnecessary-lambda
import unittest
from typing import List
Expand Down
1 change: 0 additions & 1 deletion tests_3_8/test_error.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# pylint: disable=missing-docstring
# pylint: disable=invalid-name
# pylint: disable=unused-argument
# pylint: disable=no-self-use
# pylint: disable=unused-variable

import textwrap
Expand Down
2 changes: 1 addition & 1 deletion tests_3_8/test_represent.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#!/usr/bin/env python3
# pylint: disable=missing-docstring,invalid-name,no-self-use
# pylint: disable=missing-docstring,invalid-name
# pylint: disable=unused-argument
import textwrap
import unittest
Expand Down
Loading