Skip to content
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 template download check on windows #528

Merged
merged 2 commits into from
Feb 16, 2023

Conversation

tchalabi
Copy link
Contributor

@tchalabi tchalabi commented Feb 15, 2023

following up on #520 and #499
@frandiox i am having the same issue and sadly your PR didn't solve it for me,
so i looked into the issue on my own and found this

image

after wrapping the check with path.normalize i was able to execute the command without any issue.

image

i made a PR in the hope to help others that have the same issue 🙂

if you need any additional info/help with testing debugging please feel free to ping me

best regards

@tchalabi tchalabi closed this Feb 15, 2023
@tchalabi tchalabi reopened this Feb 15, 2023
@blittle
Copy link
Contributor

blittle commented Feb 15, 2023

Thank you so much for the help @tchalabi! Sadly we are still working on getting a Windows license on our end, so your feedback really helps!

@DavidWittness
Copy link
Contributor

I verified @tchalabi 's changes on my own machine.

create-hydrogen-3-attempt

Copy link
Contributor

@frandiox frandiox left a comment

Choose a reason for hiding this comment

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

Thanks! This should fix the init command, and then we need #520 to fix the dev command.
Going to test this on Windows soon.

@frandiox frandiox merged commit 9ada30f into Shopify:2023-01 Feb 16, 2023
@github-actions github-actions bot mentioned this pull request Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants