-
Notifications
You must be signed in to change notification settings - Fork 156
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
source link the netcore pdb files #266
Conversation
@enricosada & @forki, regarding fsprojects/Paket#2930, any idea how I do the equivalent of this using paket? <ItemGroup>
<PackageReference Include="SourceLink.Create.CommandLine" Version="2.6.0" PrivateAssets="All" />
<PackageReference Include="SourceLink.Embed.PaketFiles" Version="2.6.0" PrivateAssets="All" />
</ItemGroup> Also created ctaggart/SourceLink#288. |
@enricosada Assuming CI passes and you are happy with the non-perfect Paket integration, this is good to squash merge. |
@ctaggart about |
@enricosada, sure I opened up #267 with the changes. |
Following up on #264, this adds source link info to the portable pdb files shipping for netcore.
The pdb files can be tests using
dotnet sourcelink test
:Currently getting some invalid json locally for fsautocomplete.pdb:
And then I'm running into a dotnet 2.1.2 bug that is fixed by 2.1.3, but figuring out how to update on AppVeyor.
ctaggart/SourceLink#288