Skip to content

Conversation

@jashook
Copy link
Contributor

@jashook jashook commented Jun 29, 2020

No description provided.

@jashook
Copy link
Contributor Author

jashook commented Jun 29, 2020

/cc @CarolEidt

}

[Theory]
[ActiveIssue("https://github.com/dotnet/runtime/issues/38455", TestRuntimes.CoreCLR)]
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if it's preferable to use that, or something like:

[SkipOnCoreClr("https://github.com/dotnet/runtime/issues/38455")]

Copy link
Member

Choose a reason for hiding this comment

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

Whenever there is an issue open to fix that we try to use ActiveIssue... whenever there is a reason that we don't plan to fix it because it is a long running test or it is a test stress mode that we want to disable on, we use SkipOnCoreClr.

@safern safern merged commit a034f6d into dotnet:master Jun 30, 2020
@jashook jashook deleted the disable_38455 branch June 30, 2020 18:38
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants