comments

apache · Jun 15, 2022 · 45ca55e · 45ca55e
1 parent 68ea781
commit 45ca55e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/tests/lint/pylint.sh b/tests/lint/pylint.sh
@@ -22,4 +22,8 @@ python3 -m pylint vta/python/vta --rcfile="$(dirname "$0")"/pylintrc
 python3 -m pylint tests/python/unittest/test_tvmscript_type.py --rcfile="$(dirname "$0")"/pylintrc
 python3 -m pylint tests/python/contrib/test_cmsisnn --rcfile="$(dirname "$0")"/pylintrc
 python3 -m pylint tests/python/relay/aot/*.py --rcfile="$(dirname "$0")"/pylintrc
+python3 -m pylint python/tvm --rcfile="$(dirname "$0")"/pylintrc
+python3 -m pylint vta/python/vta --rcfile="$(dirname "$0")"/pylintrc
+python3 -m pylint tests/python/unittest/test_tvmscript_type.py --rcfile="$(dirname "$0")"/pylintrc
+python3 -m pylint tests/python/contrib/test_cmsisnn --rcfile="$(dirname "$0")"/pylintrc
 python3 -m pylint tests/python/ci --rcfile="$(dirname "$0")"/pylintrc
diff --git a/tests/python/ci/test_ci.py b/tests/python/ci/test_ci.py
@@ -381,7 +381,7 @@ def test_ping_reviewers(tmpdir_factory):
     """
     reviewers_script = REPO_ROOT / "tests" / "scripts" / "ping_reviewers.py"
 
-    def run(pr, check):  # pylint: disable=invalid-name
+    def run(pull_request, check):
         git = TempGit(tmpdir_factory.mktemp("tmp_git_dir"))
         # Jenkins git is too old and doesn't have 'git init --initial-branch'
         git.run("init")
@@ -392,7 +392,7 @@ def run(pr, check):  # pylint: disable=invalid-name
             "data": {
                 "repository": {
                     "pullRequests": {
-                        "nodes": [pr],
+                        "nodes": [pull_request],
                         "edges": [],
                     }
                 }
@@ -756,15 +756,9 @@ def run(source_type, data, check):
                 "login": "person5",
             },
             "labels": [{"name": "something2"}],
-            # pylint: disable=line-too-long
-            "body": textwrap.dedent(
-                """
-                `mold` and `lld` can be a much faster alternative to `ld` from gcc. We should modify our CMakeLists.txt to detect and use these when possible. cc @person1
-
-                cc @person4
-                """
-            ),
-            # pylint: enable=line-too-long
+            "body": "`mold` and `lld` can be a much faster alternative to `ld` from gcc. "
+            "We should modify our CMakeLists.txt to detect and use these when possible. cc @person1"
+            "\n\ncc @person4",
         },
         check="would have updated issues/1234 with {'body': '\\n`mold` and `lld` can be a much"
         " faster alternative to `ld` from gcc. We should modify our CMakeLists.txt to "