Skip to content

fix: replace all separators with / #50

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 23, 2021
Merged

fix: replace all separators with / #50

merged 1 commit into from
Nov 23, 2021

Conversation

@edusperoni
Copy link
Contributor

@edusperoni edusperoni commented Jun 13, 2021

fixes issues where on windows it would resolve to:

test/a\b instead of test/a/b

@cla-bot cla-bot bot added the cla: yes label Jun 13, 2021
@edusperoni edusperoni requested a review from NathanWalker June 13, 2021 18:26
@jcassidyav
Copy link

jcassidyav commented Jul 22, 2021

@NathanWalker Could we get a release of this please.

@NathanWalker NathanWalker merged commit 4ba5ee9 into master Nov 23, 2021
@NathanWalker NathanWalker deleted the fix/windows-path branch November 23, 2021 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@NathanWalker NathanWalker Awaiting requested review from NathanWalker

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@edusperoni @jcassidyav @NathanWalker