serverless-webpack-plugin icon indicating copy to clipboard operation
serverless-webpack-plugin copied to clipboard

Support Serverless 1.0

Open StuartHarris opened this issue 9 years ago • 4 comments

Adds a new plugin to support webpack bundling for Serverless 1.0.

  • [x] add new plugin
  • [x] update README
  • [x] add npm run build to compile es6 to es5
  • [x] es6 linting support
  • [ ] write tests

StuartHarris avatar Aug 06 '16 12:08 StuartHarris

How do you propose we continue to support users of 0.5.x? Release to npm with a beta version like the core library is doing? I've been heads down lately working on a project that is running 0.5.6 so I haven't been able to check out all the new goodies in 1.0beta.

asprouse avatar Aug 08 '16 16:08 asprouse

I think we can do both. I haven't changed the 0.5 plugin, so this still works with serverless 0.5:

plugins: [
    "serverless-webpack-plugin"
]

Instead, I placed the new plugin in a 1.0 directory, so this works with serverless 1.0:

plugins:
  - serverless-webpack-plugin/1.0

Plugins are easier to write for 1.0, but are not compatible, so I needed to do something like that.

I think it would be a good idea to release with a beta version like the core library is doing. In fact it may be cool to align the version numbers with core as best we can. Maybe.

I've also been up to my eyeballs so haven't written tests yet. Hoping to do that this weekend.

StuartHarris avatar Aug 10 '16 13:08 StuartHarris

IMHO maintaining two versions in the same repository feels a bit tedious (think about two sets of node module dependencies).

I'd suggest moving 0.5 compatible stuff to 0.5 branch, and maintaining it there. When new versions are released, they could be 0.5.x. It would be almost semver compatible, considering these changes were just patches and new feature development would happen in master.

The master branch could be then kept in sync with serverless master branch, and if a 1.0 compatible version comes out and works against sls 1.0 (when it's out), then such version could be released in npm repo as well.

I think such a way would be intuitive for the developers using the package, yet it would force to follow the sls development.

laurisvan avatar Aug 23 '16 14:08 laurisvan

Hello! I'd like to use this plugin with serverless 1.0. What would be required for this to be merged in?

lpil avatar Sep 12 '16 17:09 lpil