areusch commented on pull request #8992:
URL: https://github.com/apache/tvm/pull/8992#issuecomment-918604587


   @mikepapadim this looks much closer to what we want but I think if I'm 
reading this right, I see:
   - whenever a change only modifies `docs/`, we skip all build, unit test, 
integration test steps
   
   as @Hzfengsy said, we shouldn't ever skip these nodes:
   - Sanity Check (this should already be correct in the patch now)
   - BUILD: GPU
   - docs: GPU
   
   i think we need to find a way to propagate these conditions down to the node 
level in each stage (e.g. skip each node not listed above when `changeset 
"docs/**"` doesn't match). i'm not entirely sure how to do this.


-- 
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