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
Convert path to URL before import of config/plugin for Windows support #2339
Conversation
Maybe you can add an acceptance test that passes a windows-style path to the CLI? There's a config path CLI option. |
From what I remember, we had trouble reproducing the issue in the test and so I wasn't confident that the test actually verified it was fixed. |
Is there any update on this PR? If not is there any decent workaround? |
If someone can confirm that this PR fixes the issue, we might be able to move forward with it. |
@bmish I just pulled this branch down for ember-template-lint and tested. This absolutely fixes the issue. |
33b7e7f
to
9073166
Compare
1f19911
to
03e4b93
Compare
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.
Thanks! Let's give this a try.
fixes #2336
I'm not sure how to test it, because chain for this function is quite long, should we create unit test for 'requirePlugin' function itself?