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.

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

Add reference assemblies for net6.0-[platform] TFMs #892

Closed
Youssef1313 opened this issue Aug 31, 2021 · 4 comments · Fixed by #894
Closed

Add reference assemblies for net6.0-[platform] TFMs #892

Youssef1313 opened this issue Aug 31, 2021 · 4 comments · Fixed by #894

Comments

@Youssef1313
Copy link
Member

https://github.com/dotnet/designs/blob/main/accepted/2021/net6.0-tfms/net6.0-tfms.md#design

@Youssef1313
Copy link
Member Author

@sharwell Is this possible? If yes, it would be nice to have :)

@jmarolf
Copy link
Contributor

jmarolf commented Aug 31, 2021

So the ask is to have Net60Windows, Net60Android, etc in addition to Net60 like we have here?

I have no problem with that, but it would require there to be nuget packages with these references. Currently we rely on Microsoft.NETCore.App.Ref.

This is my best understanding of the reference nuget packages and how they map. You will need to add all the reference from Microsoft.NETCore.App.Ref as well.

Property Name Nuget Package
Net60Windows Microsoft.WindowsDesktop.App.Ref
Net60Android Microsoft.Android.Ref
Net60iOS Microsoft.iOS.Ref
Net60MacOS Microsoft.macOS.Ref
Net60MacCatalyst Microsoft.MacCatalyst.Ref
Net60TvOS Microsoft.tvOS.Ref
Net60Tizen ???

@Youssef1313
Copy link
Member Author

@jmarolf Yes that's exactly what I'm asking for.
I'm not sure whether there are NuGet packages for these or not. But the lack of these reference assemblies makes testing analyzers/generators that are very domain-specific in these areas nearly impossible.

jmarolf added a commit to jmarolf/roslyn-sdk that referenced this issue Aug 31, 2021
@Youssef1313
Copy link
Member Author

@jmarolf Thanks!! That was too fast :)

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 a pull request may close this issue.

2 participants