[GitHub] [tvm] driazati commented on a diff in pull request #11258: [ci] Use r5.large nodes for builds and lint

[GitBox <gi...@apache.org>]

driazati commented on code in PR #11258:
URL: https://github.com/apache/tvm/pull/11258#discussion_r869530845


##########
tests/scripts/task_lint.sh:
##########
@@ -25,48 +25,69 @@ cleanup()
 trap cleanup 0
 
 
-echo "Convert scripts to Python..."
-tests/scripts/task_convert_scripts_to_python.sh
+# These shards are solely for CI to enable the lint job to have some parallelism.
 
-# TODO: Remove this ad-hoc pip install once https://github.com/apache/tvm/pull/10741
-# is added to the ci_lint Docker image
-python3 -m pip install --user -r jenkins/requirements.txt
-echo "Check Jenkinsfile generation"
-python3 jenkins/generate.py --check
+function shard1 {
+  echo "Convert scripts to Python..."
+  tests/scripts/task_convert_scripts_to_python.sh
 
-echo "Checking file types..."
-python3 tests/lint/check_file_type.py
+  # TODO: Remove this ad-hoc pip install once https://github.com/apache/tvm/pull/10741

Review Comment:
   Yup you're right, we don't need this anymore
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org