Skip to content

Conversation

@Mousius
Copy link
Member

@Mousius Mousius commented Mar 10, 2022

As part of this I had to disable some of the Target specific tests which didn't run under CI correctly

As part of this I had to disable some of the Target specific tests which didn't run under CI correctly
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Mousius Mousius merged commit de945e0 into main Mar 10, 2022
@Mousius Mousius deleted the more-topi branch March 10, 2022 15:55
@tkonolige
Copy link
Contributor

tkonolige commented Mar 10, 2022

@leandron @Mousius Next time you have to disable a test could you tag the person you think is responsible for the test and create a tracking issue.

Also, why is this test disabled? Is it failing? If so, why?

@areusch
Copy link
Contributor

areusch commented Mar 10, 2022

it would be great if we could capture the disabled test into a GH issue so that we can keep track!

pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
As part of this I had to disable some of the Target specific tests which didn't run under CI correctly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants