Skip to content

Suppress build warnings#40

Merged
jviau merged 1 commit intomicrosoft:mainfrom
jviau:fix-warnings
Aug 22, 2022
Merged

Suppress build warnings#40
jviau merged 1 commit intomicrosoft:mainfrom
jviau:fix-warnings

Conversation

@jviau
Copy link
Member

@jviau jviau commented Aug 17, 2022

  • Fix test project nuget package reference 0.4.0-beta -> 0.4.1-beta
  • Suppress logger message warnings - they are intentional. We want those values as dimensions, but not in the message.
  • Suppress packaging of samples
  • Suppress erroneous NU5128 with generator project via SuppressDependenciesWhenPacking

cgillum
cgillum approved these changes Aug 22, 2022
View reviewed changes
Copy link
Member

@cgillum cgillum left a comment

Choose a reason for hiding this comment

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

LGTM!

@jviau jviau merged commit fcdac96 into microsoft:main Aug 22, 2022
@jviau jviau deleted the fix-warnings branch August 22, 2022 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cgillum cgillum cgillum approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jviau @cgillum