Merge pull request #1 from MichaelRBond/lifecycle-update

Update to new lifecycle hook, `package` to remove deprecation warning

Author: ArtificerEntertainment

add-log-retention.js

@@ -16,7 +16,7 @@ class AwsAddLogRetention {
     this.options = options;
     this.provider = this.serverless.getProvider('aws');
     this.hooks = {
-      'before:deploy:createDeploymentArtifacts': this.beforeDeploy.bind(this),
+      'package:createDeploymentArtifacts': this.beforeDeploy.bind(this),
     };
   }
 

test/add-log-retention.js

@@ -37,10 +37,10 @@ describe('serverless-plugin-log-retention', function() {
       const instance = createTestInstance();
       expect(instance)
         .to.have.property('hooks')
-        .that.has.all.keys('before:deploy:createDeploymentArtifacts');
+        .that.has.all.keys('package:createDeploymentArtifacts');
 
       const stub = sinon.stub(instance, 'addLogRetentionForFunctions');
-      instance.hooks['before:deploy:createDeploymentArtifacts']();
+      instance.hooks['package:createDeploymentArtifacts']();
 
       sinon.assert.calledOnce(stub);
     });