Skip to content

attempt to fix #184 #251

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

Merged
merged 1 commit into from
Sep 20, 2018
Merged

attempt to fix #184 #251

merged 1 commit into from
Sep 20, 2018

Conversation

dschep
Copy link
Contributor

@dschep dschep commented Sep 19, 2018

Looking at the 1st time #184 was fixed, this should do the trick. Further evidence that #249 is really needed.

@rob-deans, could you try this out?

npm i unitedincome/serverless-python-requirements#win-path-spaces

@rob-deans
Copy link

Looks like it has fixed it, at least on my end, thanks!

@dschep dschep merged commit 3b2c245 into master Sep 20, 2018
@dschep
Copy link
Contributor Author

dschep commented Sep 20, 2018

I'm gonna hold off on a release until I merge or reject #253 since it's also related to file paths on windows.

@dschep dschep deleted the win-path-spaces branch September 20, 2018 14:17
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

Successfully merging this pull request may close these issues.

2 participants