leandron commented on issue #16368:
URL: https://github.com/apache/tvm/issues/16368#issuecomment-1894089436

   -1 (binding)
   
   It has been raised before that [some tests are disabled in the `unity` 
branch](https://github.com/apache/tvm/blob/a2a1b534024e677a9407bec8d09d99f2237b2b0b/ci/jenkins/generated/gpu_jenkinsfile.groovy#L57-L60),
 so it would be good to double check that as this proposal is actioned by its 
proponents, we guarantee that all current `main` tests are passing on the 
proposed merged branch. It shouldn't be an onus of contributors working on 
`main` to re-enable their tests.
   
   The linked source has a note that the test workaround shouldn't be moved 
into the target branch but the current vote statement isn't covering for this 
"disabled tests" item. If that's already planned, please let me know and I'll 
change the vote.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to