search

tmslnz / gulp-shopify-theme

Shopify theme synchronisation during development
Apache License 2.0
24 stars 6 forks source link

Windows issue #4

Closed shelldandy closed 6 years ago

shelldandy commented 7 years ago

https://github.com/tmslnz/gulp-shopify-theme/blob/master/index.js#L110

We have found that that line causes problems on Windows machines we have fixed it by replacing it with

basepath = basepath[0].replace('\\', '/');

Might be worth taking a look.

Thanks for the awesome plugin

tmslnz commented 7 years ago

Thank you for reporting this. I believe this could be solved by using the path module which handles platform-specifc formats well.

tmslnz commented 6 years ago

Fixed with commit 542d2ac444d0c3cb86b86da291f80be1304a32f3