I forgot to add that the runtime PR can actually be merged ahead of the actual change upstream which I think we should accept. Will wait 48hrs for comments before having a merge fest by silent assent.
-r On Tue, Dec 3, 2019 at 10:17 AM Rodric Rabbah <rod...@gmail.com> wrote: > I went ahead and created PRs for most of the runtimes - ones missing are > Docker, Go and Swift. > > I noticed for Java and Ruby we have two copies of mostly the same code for > action-loop and non-action-loop. Do we need two copies of the same code? > These exists in Java and Ruby. I didn't spot them elsewhere. > > https://github.com/apache/openwhisk-runtime-nodejs/pull/154 > https://github.com/apache/openwhisk-runtime-python/pull/76 > https://github.com/apache/openwhisk-runtime-php/pull/71 > https://github.com/apache/openwhisk-runtime-dotnet/pull/27 > https://github.com/apache/openwhisk-runtime-java/pull/104 > https://github.com/apache/openwhisk-runtime-java/pull/105 > https://github.com/apache/openwhisk-runtime-ruby/pull/38 > https://github.com/apache/openwhisk-runtime-ruby/pull/39 > https://github.com/apache/openwhisk-runtime-rust/pull/18 > >
