-
Notifications
You must be signed in to change notification settings - Fork 8
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
Rewrite path parsing using the path
module instead of regexes
#14
Comments
I don't believe there are tests for it. |
It turns out nodejs still does not have path.split, we cant bypass regexes entirely |
We could use something like this, but I don't particularly like the idea of a loop that I can't prove it will terminate.
|
This looks better, only that it still doesn't use the path module:
What do you think? |
@sorgloomer Needs some extra logic to support namespaced packages. |
Also, it might be easier to use |
In the recent weeks we had multiple high priority bugs, which were caused by the same reason: parsing paths with regexes.
I happily create a PR for this, but I have no idea what does the
getModuleName
function do. Are there some test cases for that?The text was updated successfully, but these errors were encountered: