-
-
Notifications
You must be signed in to change notification settings - Fork 38
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 PackageReferences to nuget packages #216
Conversation
1c61d61
to
54e1f72
Compare
Fascinating that what we had didn't capture all of the PackageReferences - maybe we aren't targeting the right design-time target or something? This seems good to add in any case. I'd be interested to see if SDK-inserted implicit PackageReferences (e.g. if you publish self contained for another RID) also get shown here. |
I'm ok with doing the one-pass check here too - perf is important and doing things in single-pass is an easy way to deal with it. |
If there's a chance of overlap, perhaps we might need to check for duplicates in the result 😨 |
54e1f72
to
30c8b5f
Compare
There are indeed overlaps, in the case of
Note the repeated FSharp.Core. |
I'm afraid I think my desired outcome ("get the path to the DLLs implied by all NuGet packages tagged with my particular custom metadata item") is beyond my current powers. The |
@Smaug123 how are you approaching this? Do you have a binlog that you're looking at? If so, it might be useful to look at the |
(I've used up basically all my extracurricular energy for a while; I may eventually come back to this, but very possibly not. I solved my immediate problem using msbuild XML instead.) |
This is an alternative phrasing of #215 . Feel free to pick either one depending on which you prefer.
I think we are currently simply missing all
PackageReference
s?There's no corresponding
NuGetPackageId
on aPackageReference
, so I've gone with simply obtaining theInclude
value.The whitespace changes earlier in the file are the result of
dotnet fantomas .
.