[Utils] Add xfail sync script#1301
Closed
bob80905 wants to merge 3 commits into
Closed
Conversation
Contributor
|
I'm not really convinced this is useful. Most of the time the first time something fails or xpasses it's pretty obvious what we should do with it, so I expect the frequency with which this gives us any extra information to be low. |
Contributor
|
I've put up #1303 which I think gives a better overview of what's currently happening and might be sufficient instead of this. |
Contributor
Author
|
Talked with Justin offline, I'll close this in favor of |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR implements a way to track the XFAILs that have been added to the offload test suite.
There are 2 failure cases this script is able to catch.
Here is the output of the script when run:

Addresses #1300
Assisted by: Github Copilot