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

Use Microsoft.SourceLink.GitHub #6421

Merged
merged 3 commits into from
Dec 10, 2020
Merged

Use Microsoft.SourceLink.GitHub #6421

merged 3 commits into from
Dec 10, 2020

Conversation

maliming
Copy link
Member

@maliming maliming commented Dec 1, 2020

Fix #6398 fix #3462

image

@maliming maliming added this to the 4.1-preview milestone Dec 1, 2020
@maliming maliming marked this pull request as draft December 1, 2020 09:10
@maliming maliming marked this pull request as ready for review December 1, 2020 10:31
@realLiangshiwei realLiangshiwei merged commit f1550e5 into dev Dec 10, 2020
@realLiangshiwei realLiangshiwei deleted the maliming/sourcelink branch December 10, 2020 03:17
@hikalkan
Copy link
Member

Can you test this with nightly nuget packages to see if it is working?

@hikalkan
Copy link
Member

I created a console app with 4.0, used switch-to-preview to update to 4.1.0-previewXXX but could not debug it. I enabled debugging in VS, but couldn't make it possible. I didn't spend much time to understand the reason, can you please check it.

@maliming
Copy link
Member Author

I embedded the PDB files into the NuGet package, but dotnet did not copy these PDB files into the project.

Developers can use https://www.nuget.org/packages/SourceLink.Copy.PdbFiles to solve this problem.

We should publish the symbol package to nuget. I will work on it.

dotnet/sdk#1458

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use Microsoft.SourceLink.GitHub source link broken
3 participants