Skip to content
New issue

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

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

Already on GitHub? # to your account

Add support for threading MSBuild item metadata to analyzer options #3950

Merged
merged 2 commits into from
Aug 4, 2020

Conversation

mavasani
Copy link
Contributor

@mavasani mavasani commented Aug 4, 2020

Addresses analyzer part of dotnet/sdk#12706
Adds support for platform compat check analyzer to be able to read SupportedPlatform items.

Analyzers should invoke context.Options.GetMSBuildItemMetadataValues extension method to query this option.

User project file

image

Analyzer options with item metadata

image

Addresses analyzer part of dotnet/sdk#12706
Adds support for platform compat check analyzer to be able to read `SupportedPlatform` items.
Copy link
Contributor

@buyaa-n buyaa-n left a comment

Choose a reason for hiding this comment

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

Left few NIT, overall looks good, thanks!

Co-authored-by: Buyaa <buyankhishig.namnan@microsoft.com>
@codecov
Copy link

codecov bot commented Aug 4, 2020

Codecov Report

Merging #3950 into master will decrease coverage by 0.00%.
The diff coverage is 13.33%.

@@            Coverage Diff             @@
##           master    #3950      +/-   ##
==========================================
- Coverage   95.64%   95.63%   -0.01%     
==========================================
  Files        1153     1155       +2     
  Lines      254404   254759     +355     
  Branches    15242    15252      +10     
==========================================
+ Hits       243326   243649     +323     
- Misses       9139     9161      +22     
- Partials     1939     1949      +10     

@mavasani
Copy link
Contributor Author

mavasani commented Aug 4, 2020

Thanks @buyaa-n!

@mavasani mavasani merged commit 31fe7d4 into dotnet:master Aug 4, 2020
@mavasani mavasani deleted the MSBuildItemNames branch August 4, 2020 22:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants