Add support for multiple runtimes in service

[brendanhonea]

What did you implement:

Added support for projects that have lambdas running on multiple runtimes. In our case, we needed some java lambdas to run in the same project as our typescript lambdas and the plugin was trying to bundle all runtimes.

How did you implement it:

In validate.js, added a check for runtime.startswith('node') so that the plugin only tries to package node lambdas.

How can we verify it:

Create a serverless project that includes node lambdas, and lambdas from other supported runtimes (java, python, etc). Make sure that package.invidually is true. Master branch will try to package the other runtimes with webpack and will cause errors when invoking the lambdas. This fork will package only node lambdas with the plugin

Todos:

• Write tests
• Write documentation (None needed)
• Fix linting errors
• Make sure code coverage hasn't dropped
• Provide verification config / commands / resources
• Enable "Allow edits from maintainers" for this PR
• Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

[ineffyble]

Any updates on this? It looks like an alternative to #256

[somq]

Hi @brendanhonea, thanks for the PR.
Would be nice to also add a skipCompile like flag here in the before:invoke:local:invoke hook to allow sls invoke local to work with Python lambdas

      'before:invoke:local:invoke': () =>
          ...

            const functionDef = this.serverless.service.getFunction(this.options.function);
            this.isFuncNode = _.startsWith(functionDef.runtime, 'node')

            return this.serverless.pluginManager.spawn('webpack:validate');
          })
          .then(() => (this.skipCompile || !this.isFuncNode ? BbPromise.resolve() : this.serverless.pluginManager.spawn('webpack:compile')))
          .then(() => (this.isFuncNode ? this.prepareLocalInvoke() : BbPromise.resolve())),

Could you eventually add this to your fork?

[miguel-a-calles-mba]

What happens when the global runtime is node and the runtime is not specified per function?

[miguel-a-calles-mba]

Closing. Will use this PR instead: Check for node runtimes first #579.