dfinity / dfxvm

dfx version manager
Apache License 2.0
7 stars 5 forks source link

refactor: construct Locations in one place #40

Closed ericswanson-dfinity closed 8 months ago

ericswanson-dfinity commented 8 months ago

Description

All of the dfxvm commands need the Locations object. self update needs it a little higher in the call stack, in order to delete a binary.

This PR makes it so there is only one place that calls Locations::new(), and it's higher in the call stack. This also means there are fewer places that can return NoHomeDirectory as an error.

How Has This Been Tested?

Covered by CI