Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Replace uses of Xunit.PlatformID with renamed TestPlatforms#12284

Merged
stephentoub merged 1 commit into
dotnet:masterfrom
justinvp:testplatforms
Oct 2, 2016
Merged

Replace uses of Xunit.PlatformID with renamed TestPlatforms#12284
stephentoub merged 1 commit into
dotnet:masterfrom
justinvp:testplatforms

Conversation

@justinvp
Copy link
Copy Markdown
Contributor

@justinvp justinvp commented Oct 1, 2016

Replace uses of Xunit.PlatformID with renamed TestPlatforms because the old name conflicts with System.PlatformID and should be plural according to the design guidelines for flags enums.

Reference:
#12267

dotnet/buildtools#999
dotnet/buildtools#1000

Replace uses of `Xunit.PlatformID` with renamed `TestPlatforms` because
the old name conflicts with `System.PlatformID` and should be plural
according to the design guidelines for flags enums.
Copy link
Copy Markdown
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

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

Thanks!

@stephentoub stephentoub merged commit 3d3c860 into dotnet:master Oct 2, 2016
@justinvp justinvp deleted the testplatforms branch October 2, 2016 15:58
@justinvp
Copy link
Copy Markdown
Contributor Author

justinvp commented Oct 2, 2016

Thanks, @stephentoub! cc: @weshaggard as FYI

@weshaggard
Copy link
Copy Markdown
Member

Thanks @justinvp

@karelz karelz modified the milestone: 1.2.0 Dec 3, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Replace uses of Xunit.PlatformID with renamed TestPlatforms

Commit migrated from dotnet/corefx@3d3c860
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants