comments

apache · Jun 15, 2022 · 4035ab9 · 4035ab9
1 parent b330d8c
commit 4035ab9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 15 deletions.
diff --git a/tests/lint/pylint.sh b/tests/lint/pylint.sh
@@ -17,9 +17,9 @@
 # under the License.
 set -euxo pipefail
 
-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 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
@@ -346,7 +346,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")
@@ -357,7 +357,7 @@ def run(pr, check):  # pylint: disable=invalid-name
             "data": {
                 "repository": {
                     "pullRequests": {
-                        "nodes": [pr],
+                        "nodes": [pull_request],
                         "edges": [],
                     }
                 }
@@ -721,15 +721,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 "