-
Notifications
You must be signed in to change notification settings - Fork 886
fix: fix coder template pull on Windows #9327
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
Conversation
Signed-off-by: Spike Curtis <spike@coder.com>
Unfortunately, extracting the tests out of a |
Signed-off-by: Spike Curtis <spike@coder.com>
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.
👍 as along as CI is happy
Using the hide whitespace feature made it quite alright 👌 |
Signed-off-by: Spike Curtis <spike@coder.com>
Tested here seems to resolve the issue. PS C:\Users\matif\Downloads> .\coder_test.exe login https://pr9327.test.cdr.dev/
Your browser has been opened to visit:
https://pr9327.test.cdr.dev/cli-auth
> Paste your token here:
> Welcome to Coder, coder! You're authenticated.
PS C:\Users\matif\Downloads> .\coder_test.exe templates ls
NAME LAST UPDATED USED BY
kubernetes August 25, 2023 - active developers
PS C:\Users\matif\Downloads> .\coder_test.exe templates pull kubernetes
Extracting template to "kubernetes"
PS C:\Users\matif\Downloads> ls .\kubernetes\
Directory: C:\Users\matif\Downloads\kubernetes
Mode LastWriteTime Length Name
---- ------------- ------ ----
-a--- 25/08/2023 12:16 pm 2489 .terraform.lock.hcl
-a--- 25/08/2023 12:16 pm 8147 main.tf |
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.
tested in a PR deployment and seems to resolve the issue.
…-template-pull-windows
fixes #9291 and adds a unit test