-
Notifications
You must be signed in to change notification settings - Fork 164
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
fix: windows CLI is installed in wrong location #420
fix: windows CLI is installed in wrong location #420
Conversation
Before this commit, when installing the CLI using this script, it would move the downloaded CLI executable as a file to `%LOCALAPPDATA%/Appwrite` while it is inteded to move it to `%LOCALAPPDATA%/Appwrite/appwrite.exe`. This commit fixes this problem by creating the directory explicitly and moving the file into it.
@TorstenDittmann Can you give this a review ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Template looks good to me.
I'm not well versed into windows shell, so Im going to let @TorstenDittmann help me there
Awesome work @benjaminSchilling33
Thanks @christyjacob4 In the meanwhile this seems to bei redundant to #397 |
Co-authored-by: Christy Jacob <christyjacob4@gmail.com>
Thanks for the PR 🥳 It will be part of the next release 🙂 |
Before this commit, when installing the CLI using this script, it would move the downloaded CLI executable
as a file to
%LOCALAPPDATA%/Appwrite
while it is intended to move it to%LOCALAPPDATA%/Appwrite/appwrite.exe
.This commit fixes this problem by creating the directory explicitly and moving the file into it.
closes: appwrite/appwrite#3007