Skip to content

Rewrord error message on failed init#131

Merged
dgrove-oss merged 4 commits into
apache:masterfrom
dengliming:patch-1
Sep 25, 2020
Merged

Rewrord error message on failed init#131
dgrove-oss merged 4 commits into
apache:masterfrom
dengliming:patch-1

Conversation

@dengliming
Copy link
Copy Markdown
Member

Fixes #125

@rabbah
Copy link
Copy Markdown
Member

rabbah commented Sep 12, 2020

Thanks @dengliming for the contribution 🎉

@dgrove-oss
Copy link
Copy Markdown
Member

I think you also need to update the expected test output at https://github.com/apache/openwhisk-runtime-go/blob/master/openwhisk/executor_test.go#L108 to match the changed error message.

@dengliming
Copy link
Copy Markdown
Member Author

@dgrove-oss Thanks. Fixed at lastest commit.

@dgrove-oss dgrove-oss merged commit e6e45e1 into apache:master Sep 25, 2020
falkzoll added a commit to falkzoll/incubator-openwhisk-runtime-python that referenced this pull request Sep 28, 2020
- Update to golang:1.15 to continue to get security fixes.
- Update from debian stretch to buster stay at latest stable to continue to get security fixes.
- Fix testcase in PythonActionLoopExtraTests.scala due to changed wording in the go runtime (apache/openwhisk-runtime-go#131).
mhenke1 pushed a commit to apache/openwhisk-runtime-python that referenced this pull request Sep 28, 2020
- Update to golang:1.15 to continue to get security fixes.
- Update from debian stretch to buster stay at latest stable to continue to get security fixes.
- Fix testcase in PythonActionLoopExtraTests.scala due to changed wording in the go runtime (apache/openwhisk-runtime-go#131).
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.

rewrord error message on failed init

3 participants