Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

Commit 987dea5

Browse files
committed
post-review updates
- Ensure that the deploy prompt can be seen.
1 parent 20f2b17 commit 987dea5

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

gulpfile.js

+3-4
Original file line numberDiff line numberDiff line change
@@ -890,6 +890,7 @@ function harpCompile() {
890890
spawnInfo.promise.then(function(x) {
891891
gutil.log("NODE_ENV: " + process.env.NODE_ENV);
892892
showHideExampleNodeModules('show');
893+
showHideApiDir('show');
893894
if (x !== 0) {
894895
deferred.reject(x)
895896
} else {
@@ -899,9 +900,8 @@ function harpCompile() {
899900
}).catch(function(e) {
900901
gutil.log("NODE_ENV: " + process.env.NODE_ENV);
901902
showHideExampleNodeModules('show');
902-
deferred.reject(e);
903-
}).finally(() => {
904903
showHideApiDir('show');
904+
deferred.reject(e);
905905
});
906906
return deferred.promise;
907907
}
@@ -1148,11 +1148,10 @@ function watchAndSync(options, cb) {
11481148

11491149
// returns a promise;
11501150
function askDeploy() {
1151-
11521151
prompt.start();
11531152
var schema = {
11541153
name: 'shouldDeploy',
1155-
description: 'Deploy to Firebase? (y/n): ',
1154+
description: 'Deploy to Firebase? (y/n)',
11561155
type: 'string',
11571156
pattern: /Y|N|y|n/,
11581157
message: "Respond with either a 'y' or 'n'",

0 commit comments

Comments
 (0)