Skip to content

Conversation

@sciabarracom
Copy link
Contributor

@sciabarracom sciabarracom commented Jul 6, 2021

Note support for requirements.txt requires an update to the goproxy

apache/openwhisk-runtime-go#148

Copy link
Member

@rabbah rabbah left a 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.

Luke-Roy-IBM and others added 5 commits July 7, 2021 12:49
- 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>
Copy link
Member

@rabbah rabbah left a 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.

@sciabarracom
Copy link
Contributor Author

plase merge #111

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.

normalize naming to remove actionLoop Setting requirements for Python

5 participants