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

Does not work as intended on Windows #10

Closed
haakonmt opened this issue May 18, 2022 · 1 comment
Closed

Does not work as intended on Windows #10

haakonmt opened this issue May 18, 2022 · 1 comment

Comments

@haakonmt
Copy link

remix routes returns a different (and erronous) result when run on Windows vs. a unix-based environment.

It looks like a simple path separator issue, where the library assumes /, but Windows uses \. Can most likely be mitigated by using path.sep in the right places instead.

@kiliman
Copy link
Owner

kiliman commented May 18, 2022

Thanks for the bug report. I'm in the process of rewriting the route parser. I will make sure to update it to account for differences in path separators.

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

No branches or pull requests

2 participants