AOT compatible: NuGet.DependencyResolver.Core#7138
Conversation
zivkan
left a comment
There was a problem hiding this comment.
Is there a reason you created one PR per project?
If it was to see which ones build and which ones don't, you could have gotten quicker feedback by building locally. This property only enables analyzers, it doesn't change runtime.
That makes sense. In the previous PR #7097, we ran into an analyzer error that only reproduced on Linux, which influenced how I approached it. That said, I could have grouped the changes into a single PR as well. There wasn’t a strong technical reason for splitting them. at the time, I just chose to go project by project. |
Bug
Fixes:
Related: NuGet/Home#14408
Description
Add IsAotCompatible property and fix any AOT analysis warnings. This is part of a broader effort to make
NuGet's libraries compatible with Native AOT
PR Checklist