Skip to content

Conversation

@agocke
Copy link
Member

@agocke agocke commented Jun 25, 2019

No description provided.

@agocke agocke marked this pull request as ready for review June 26, 2019 17:10
@agocke agocke requested review from RikkiGibson and chsienki June 26, 2019 17:10
@agocke
Copy link
Member Author

agocke commented Jun 27, 2019

@RikkiGibson Could you take a look at this when you get a chance?


# Dotnet diagnostic settings
[*.{cs,vb}]
dotnet_diagnostic.xUnit2018.severity = suppress # "do not compare an object's exact type to the abstract class" is a valid assert, but very noisy right now
Copy link
Member

Choose a reason for hiding this comment

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

I assume this fixes a bunch of spurious warnings in the build?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, this is reacting to a change where we renamed "suppress" in editorconfig to "none"

<MicrosoftInternalVisualStudioShellInterop140DesignTimeVersion>14.3.25407-alpha</MicrosoftInternalVisualStudioShellInterop140DesignTimeVersion>
<MicrosoftMetadataVisualizerVersion>1.0.0-beta1-63011-01</MicrosoftMetadataVisualizerVersion>
<MicrosoftMSXMLVersion>8.0.0.0-alpha</MicrosoftMSXMLVersion>
<MicrosoftNetCompilersToolsetVersion>3.2.0-beta4-19312-15</MicrosoftNetCompilersToolsetVersion>
Copy link
Member

Choose a reason for hiding this comment

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

So the point is, let's use the same compiler to build Roslyn as we give to everyone else via Arcade?

Copy link
Member Author

Choose a reason for hiding this comment

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

Exactly.

@agocke agocke closed this Jul 1, 2019
@agocke agocke reopened this Jul 1, 2019
@agocke agocke merged commit a4cd624 into dotnet:master Jul 1, 2019
@agocke agocke deleted the use-arcade-toolset branch July 1, 2019 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants