-
Notifications
You must be signed in to change notification settings - Fork 24
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.
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
Install command #87
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Details
Name:
install
Alias:
i
Arguments:
-g, --global
) and local vr-powered installationsdeno install
(except for--root
,--force
,--reload
)Action
Adding a dependency
We should find a way to safely distinguish the two cases (ie presence of the
-g
flag or remote URLs?) and generate and execute adeno install
command either from the parameters in case 2 or from the remote vr configuration (install
entry) in case 1.Updating a dependency
If a compatible entry is found in the local vr configuration, the dependency is not installed unless a corresponding executable is not present in the local bins folder or unless the
-r, --reload
flag is used. If the entries differ by the registry or version, a new install is always performed (a@latest
version tag could be used to forcefully install the latest version of a dep?).Initializing dependencies from an existing configuration
Like
npm
, runningvr install
without arguments installs all the dependencies found in the configuration locally. If a bin folder is already present, it checks wether all the deps have a corresponding executable and installs it otherwise.Security considerations
Since in case 1 the permissions are hidden in the remote configuration, it might be appropriate to at least show the final
deno install
command that is being executed (with or without a confirmation prompt?) so that the user can decide to uninstall/reinstall the CLI with a custom install command.Always create a lockfile for these installs?
The text was updated successfully, but these errors were encountered: