Skip to content

Commit 8058863

Browse files
committed
Merge pull request #2 from lukemelia/configure-hook
Use new `configure` hook instead of `willDeploy`
2 parents 1de0c82 + 7163f4c commit 8058863

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ module.exports = {
7575
return {
7676
name: options.name,
7777

78-
willDeploy: function(context) {
78+
configure: function(context) {
7979
var deployment = context.deployment;
8080
var ui = deployment.ui;
8181
var config = deployment.config[this.name] = deployment.config[this.name] || {};

tests/unit/index-nodetest.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@ describe('gzip plugin', function() {
2626
name: 'test-plugin'
2727
});
2828

29-
assert.equal(typeof result.willDeploy, 'function');
29+
assert.equal(typeof result.configure, 'function');
3030
assert.equal(typeof result.willUpload, 'function');
3131
});
3232

33-
describe('willDeploy hook', function() {
33+
describe('configure hook', function() {
3434
it('resolves if config is ok', function() {
3535
var plugin = subject.createDeployPlugin({
3636
name: 'gzip'
@@ -47,7 +47,7 @@ describe('gzip plugin', function() {
4747
}
4848
};
4949

50-
return assert.isFulfilled(plugin.willDeploy.call(plugin, context))
50+
return assert.isFulfilled(plugin.configure.call(plugin, context))
5151
});
5252
});
5353

0 commit comments

Comments
 (0)