-
Notifications
You must be signed in to change notification settings - Fork 69
Support python39 #110
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
Support python39 #110
Conversation
rabbah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is also fixing a bug when __OW_EXECUTION_ENV is not defined and addressing venv activate on windows. It would be better if those were separated into their own PRs IMO but willing to merge them all at once.
- An akka upgrade in apache/openwhisk required changes here to successfully build and run the test cases. - fix fix taken from [openwhisk-runtime-docker 89](apache/openwhisk-runtime-docker#89)
Co-authored-by: rodric rabbah <rodric@gmail.com>
Co-authored-by: rodric rabbah <rodric@gmail.com>
tests/src/test/scala/runtime/actionContainers/Python39ctionLoopContainerTests.scala
Outdated
Show resolved
Hide resolved
rabbah
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but some of the "release" to "source" references will need to be changed back at some point when the go proxy is released.
cc492a2 to
7bdec45
Compare
|
plase merge #111 |
Note support for requirements.txt requires an update to the goproxy
apache/openwhisk-runtime-go#148