-
Notifications
You must be signed in to change notification settings - Fork 226
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
Using full file URL when importing migrations - Fixes #201 #202
Using full file URL when importing migrations - Fixes #201 #202
Conversation
Ah the joy of |
Sweet. Thanks! I noticed the github actions failing during the lint step on files that weren't touched - something to do with spaces at the beginning of some files. That's pre-existing so haven't touched it. Let me know if you need help to fix & that/what the deal is with the lint/tests. Will keep an eye out on the patch version. |
No worries on the lint failures, I agree they look unrelated and it worked fine locally. I made a comment on another PR which was near ready to merge (#200) but looks like they didn't respond so I guess I can just push the release. |
Oh, and thanks for the great contribution! |
published as |
Fixes #201
Updated the dynamic import file path to use File URL (which also works on windows)
e.g. Instead of:
it's importing: