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.

Sign up for GitHub

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

Jump to bottom

Add variable expansion in defaultDestination #759

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add variable expansion in defaultDestination #759

wants to merge 1 commit into from

Commits on Aug 4, 2024

  1. Add variable expansion in defaultDestination 

    This implements aspnet#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.
    @jimmylewis
    jimmylewis committed Aug 4, 2024
    Configuration menu
    Copy the full SHA
    d252a09 View commit details
    Browse the repository at this point in the history