aspnet / LibraryManager

MIT License
457 stars 80 forks source link

Change Manifest.InstallLibraryAsync to return a single result #737

Closed jimmylewis closed 5 months ago

jimmylewis commented 5 months ago

Came across this while refactoring other code, it was isolated enough to pull into its own PR.

This method was designed to return many results, but that doesn't make a lot of sense when installing a single library (one library => one result). Also, the existing code paths only lever returned an enumerable of one item, which is wasteful but confirms that there isn't a need for an IEnumerable here.