Skip to content

Make the duplicate usings warnings go away #34324

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

davidfowl
Copy link
Member

@davidfowl davidfowl commented Jul 14, 2021

  • The value is low and they clash in vs 2019 16.10/11
  • We can remove this when vs 2022 ships

- The value is low and they clash in vs 16.10/11
@davidfowl davidfowl requested a review from Pilchie as a code owner July 14, 2021 01:18
@davidfowl davidfowl requested review from dougbu and Tratcher July 14, 2021 01:27
@@ -111,6 +111,7 @@
<PropertyGroup>
<!-- Ensure API docs are available. -->
<NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
<NoWarn>$(NoWarn),0105</NoWarn>
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't see these warnings in rolling builds of 'main'. Could this be conditional on '$(BuildingInsideVisualStudio)' == 'true'

Also concerned about doing this everywhere. How many projects or (maybe better) test asset directories need this suppression❔

Copy link
Contributor

Choose a reason for hiding this comment

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

These are semi-colon delimited no?

Copy link
Member

Choose a reason for hiding this comment

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

Also concerned about doing this everywhere. How many projects or (maybe better) test asset directories need this suppression❔

Just in middleware.snlf I get 174 errors across 25 projects, most of which are product assemblies.

@davidfowl
Copy link
Member Author

Replaced by #34346

@davidfowl davidfowl closed this Jul 14, 2021
@davidfowl davidfowl deleted the davidfowl/nowarn-duplicate-usings branch July 25, 2021 07:49
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