Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Referencing the analyzer package with PrivateAssets="All" #1

Open
dbolin opened this issue May 10, 2019 · 0 comments
Open

Referencing the analyzer package with PrivateAssets="All" #1

dbolin opened this issue May 10, 2019 · 0 comments
Assignees
@dbolin
Labels
bug Something isn't working immutable analyzer Related to immutable analyzer

Comments

@dbolin
Copy link
Owner

dbolin commented May 10, 2019

Currently you have to reference the analyzer with PrivateAssets="All" to prevent the analyzer from flowing to referencing projects. However, this means referencing projects don't reference the Attributes package, which can cause issues with runtime reflection.

PrivateAssets="analyzers" would be the preferred solution but needs NuGet/Home#6279 to be resolved.

One workaround is to reference the Attributes package directly in projects that need it.
@dbolin dbolin added bug Something isn't working immutable analyzer Related to immutable analyzer labels May 10, 2019
@dbolin dbolin self-assigned this May 10, 2019
dbolin pushed a commit that referenced this issue Apr 4, 2020
@mwelsh1118
Merge pull request #1 from dbolin/master
1231cdf 
Merge from dbolin master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dbolin dbolin

Labels
bug Something isn't working immutable analyzer Related to immutable analyzer
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dbolin