aspnet / LibraryManager

MIT License
457 stars 80 forks source link

Add variable expansion in defaultDestination #759

Closed jimmylewis closed 1 week ago

jimmylewis commented 4 months ago

This resolves #68, except I used [Name] instead of [LibraryName]. It felt like LibraryName should be matched with LibraryVersion and that felt verbose, so I took the shorter versions.

This expansion is applied when we expand the ManifestOnDisk (which is either read from disk or from a raw JSON) into LibraryInstallationState. This is where we determine to use the defaultDestination or a library-specific destination, so it should be the only place this expansion needs to occur.