Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

WIP: Added wick install #388

Merged
merged 4 commits into from
Jul 31, 2023
Merged

WIP: Added wick install #388

merged 4 commits into from
Jul 31, 2023

Conversation

jsoverson
Copy link
Contributor

This PR adds a wick install command that creates an executable in ~/.wick/bin to install and run applications as you would other commands.

On windows: wick creates an app-name.cmd and an app-name.ps1 file to issue a wick run on the cached application config.
On non-windows: wick creates an executable text file with a /bin/sh hashbang that issues a wick run command on the cached application config.

This is WIP while I write tests.


wick run {}
"#,
path.display()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be the registry path if that exists? This would allow for running a latest and keep that automatically updated.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, i don't think so. No app should auto-update itself by default once installed on a local filesystem. It would be a bad user experience to suddenly change or break an application without giving the user a chance to prevent it.

@jsoverson jsoverson marked this pull request as ready for review July 31, 2023 18:41
@fawadasaurus fawadasaurus merged commit 3158048 into main Jul 31, 2023
@fawadasaurus fawadasaurus deleted the wick-install branch July 31, 2023 19:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants