-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
upgrade project tooling to .NET Core 3.1 #2089
Conversation
shiftkey
commented
Feb 9, 2020
•
edited
Loading
edited
- Appveyor passes
- Actions passes
- tidy up about global tools
- review diff
a63724a
to
33835ba
Compare
Codecov Report
@@ Coverage Diff @@
## master #2089 +/- ##
=======================================
Coverage 67.06% 67.06%
=======================================
Files 538 538
Lines 14138 14138
=======================================
Hits 9481 9481
Misses 4657 4657 |
f15c63c
to
75cb523
Compare
release_notes: upgraded project tooling to .NET Core 3.1 |
<Project> | ||
<PropertyGroup> | ||
<EmbedUntrackedSources>true</EmbedUntrackedSources> | ||
<TargetFrameworkMonikerAssemblyAttributesPath>$([System.IO.Path]::Combine('$(IntermediateOutputPath)','$(TargetFrameworkMoniker).AssemblyAttributes$(DefaultLanguageSourceExtension)'))</TargetFrameworkMonikerAssemblyAttributesPath> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of this, you can use https://www.nuget.org/packages/Microsoft.NETFramework.ReferenceAssemblies/
for compiling with mono.
https://stu.dev/dotnet-framework-reference-assemblies-via-nuget/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm already using Microsoft.NETFramework.ReferenceAssemblies
in this project, and this workaround was due to an upgrade issue that I uncovered related to .NET Core 3.1 and Sourcelink.
The details are over here, if you wanted to follow along: shiftkey/dotnetcore-sourcelink-test-bug#1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Neat 👍
Thanks for sharing.