Merge pull request #1 from MichaelRBond/lifecycle-update

Update to new lifecycle hook, `package` to remove deprecation warning
serverless · Aug 3, 2019 · eee52a5 · eee52a5
2 parents 091375d + 099e063
commit eee52a5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/add-log-retention.js b/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),
     };
   }
 

diff --git a/test/add-log-retention.js b/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);
     });