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

fix: Address unknown path format error in wsl2 #667

Merged

Conversation

snicmakino
Copy link
Contributor

@snicmakino snicmakino commented Jan 25, 2022

This change solves the problem #371 that is occurring in wsl2.
In order to make wsl2 work as well as linux, the wsl decision was changed so that it only applies to wsl1.

Copy link
Contributor

@pgrzesik pgrzesik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @snicmakino - thanks a lot for this, could you please address the formatting issues before merging?

@snicmakino
Copy link
Contributor Author

Hello @pgrzesik

Hello @snicmakino - thanks a lot for this, could you please address the formatting issues before merging?

Thank you for pointing that out.
I fixed it and I confirmed that prettier-check:updated is fine.

@snicmakino snicmakino requested a review from pgrzesik January 26, 2022 10:05
Copy link
Contributor

@pgrzesik pgrzesik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @snicmakino 👍

@pgrzesik pgrzesik added the bug label Jan 26, 2022
@pgrzesik pgrzesik changed the title Fix unknown path format error in wsl2 fix: Address unknown path format error in wsl2 Jan 26, 2022
@pgrzesik pgrzesik merged commit b16c82d into serverless:master Jan 26, 2022
@EnzoMolion
Copy link

Hi guys!

First, thanks for the amazing tool that is Serverless, it is much appreciated

Do you, by any chance, have an idea of when this fix will be shipped in a release?

Thanks

@pgrzesik
Copy link
Contributor

Hey @EnzoMolion - I was planning to do a release either today or early next week.

@EnzoMolion
Copy link

Ok, thanks a lot

@EnzoMolion
Copy link

Tested it today, it works like a charm, thanks @snicmakino and @pgrzesik

kaochiuan pushed a commit to kaochiuan/serverless-python-requirements that referenced this pull request Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants