You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by jo...@apache.org on 2023/10/09 18:51:53 UTC

[superset] branch master updated: chore: Update pylint to 2.17.7 (#25566)

This is an automated email from the ASF dual-hosted git repository.

johnbodley pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 42d0474cc2 chore: Update pylint to 2.17.7 (#25566)
42d0474cc2 is described below

commit 42d0474cc2fa45f6876c63edb0bfd79d7d34c48c
Author: EugeneTorap <ev...@gmail.com>
AuthorDate: Mon Oct 9 21:51:41 2023 +0300

    chore: Update pylint to 2.17.7 (#25566)
---
 .pylintrc                    | 1 -
 requirements/development.txt | 4 ++--
 superset/cli/test_db.py      | 1 -
 superset/security/manager.py | 2 +-
 superset/views/core.py       | 2 +-
 5 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/.pylintrc b/.pylintrc
index 5de0030192..6083060624 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -75,7 +75,6 @@ enable=
 # --disable=W"
 disable=
     cyclic-import,  # re-enable once this no longer raises false positives
-    no-member,  # re-enable once this no longer raises false positives. This will become redundant after the min required version is 3.11
     missing-docstring,
     duplicate-code,
     unspecified-encoding,
diff --git a/requirements/development.txt b/requirements/development.txt
index 8eea1ae4ca..447ba634aa 100644
--- a/requirements/development.txt
+++ b/requirements/development.txt
@@ -12,7 +12,7 @@
     #   -r requirements/development.in
 appnope==0.1.3
     # via ipython
-astroid==2.15.6
+astroid==2.15.8
     # via pylint
 asttokens==2.2.1
     # via stack-data
@@ -100,7 +100,7 @@ pyhive[hive_pure_sasl]==0.7.0
     # via apache-superset
 pyinstrument==4.4.0
     # via -r requirements/development.in
-pylint==2.17.4
+pylint==2.17.7
     # via -r requirements/development.in
 python-ldap==3.4.3
     # via -r requirements/development.in
diff --git a/superset/cli/test_db.py b/superset/cli/test_db.py
index b64c36156c..a7361a0d3e 100644
--- a/superset/cli/test_db.py
+++ b/superset/cli/test_db.py
@@ -329,7 +329,6 @@ def test_sqlalchemy_dialect(
     return engine
 
 
-# pylint: disable=too-many-statements
 def test_database_connectivity(console: Console, engine: Engine) -> None:
     """
     Tests the DB API 2.0 driver.
diff --git a/superset/security/manager.py b/superset/security/manager.py
index 22769591ab..dc5e76e18b 100644
--- a/superset/security/manager.py
+++ b/superset/security/manager.py
@@ -1339,7 +1339,7 @@ class SupersetSecurityManager(  # pylint: disable=too-many-public-methods
         from superset.connectors.sqla.models import SqlaTable
 
         # Check if watched fields have changed
-        table = SqlaTable.__table__
+        table = SqlaTable.__table__  # pylint: disable=no-member
         current_dataset = connection.execute(
             table.select().where(table.c.id == target.id)
         ).one()
diff --git a/superset/views/core.py b/superset/views/core.py
index 95636de6ad..2f9b99eba0 100755
--- a/superset/views/core.py
+++ b/superset/views/core.py
@@ -14,7 +14,7 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
-# pylint: disable=too-many-lines, invalid-name
+# pylint: disable=invalid-name
 from __future__ import annotations
 
 import contextlib