Convert gulp file to use runSequence and match gulp 4 practices (This fixes race condition where minify task may run before compile task finishes).

Author: mattmazzola

dist/powerbi.d.ts

@@ -117,4 +117,5 @@ export class Tile extends Embed {
     static attribute: string;
     constructor(element: HTMLElement, options: IEmbedOptions);
     getEmbedUrl(): string;
-}
+}
+

dist/powerbi.js.map

@@ -11,20 +11,53 @@ var rename = require('gulp-rename'),
     karma = require('karma'),
     paths = require('./paths'),
     webpack = require('webpack-stream'),
-    webpackConfig = require('./webpack.config')
+    webpackConfig = require('./webpack.config'),
+    runSequence = require('run-sequence')
     ;
 
 gulp.task('watch', 'Watches for changes', ['lint'], function () {
     gulp.watch(paths.jsSource, ['lint:ts']);
     gulp.watch(paths.specSource, ['lint:spec', 'test']);
 });
 
-gulp.task('lint', 'Lints all files', ['lint:ts', 'lint:spec']);
-gulp.task('test', 'Runs all tests', ['test:js', 'copy']);
-gulp.task('build', 'Runs a full build', ['build:release']);
+gulp.task('lint', 'Lints all files', function (done) {
+    runSequence(
+        ['lint:ts', 'lint:spec'],
+        done
+    );
+});
+gulp.task('test', 'Runs all tests', function (done) {
+    runSequence(
+        'clean',
+        'compile:ts',
+        ['test:js', 'copy', 'min:js'],
+        done
+    );
+});
+gulp.task('build', 'Runs a full build', function (done) {
+    runSequence(
+        'build:release',
+        done
+    );
+});
 
-gulp.task('build:debug', 'Runs a debug build', ['lint', 'compile:ts', 'copy']);
-gulp.task('build:release', 'Runs a release build', ['lint', 'compile:ts', 'copy', 'min:js']);
+gulp.task('build:debug', 'Runs a debug build', function (done) {
+    runSequence(
+        'lint',
+        'clean',
+        ['compile:ts', 'copy'],
+        done
+    );
+});
+gulp.task('build:release', 'Runs a release build', function (done) {
+    runSequence(
+        'lint',
+        'clean',
+        ['compile:ts', 'copy'],
+        'min:js',
+        done
+    )
+});
 
 gulp.task('clean', 'Cleans destination folder', function(done) {
     rimraf(paths.jsDest, done);
@@ -58,14 +91,14 @@ gulp.task('min:js', 'Creates minified JavaScript file', function() {
         .pipe(gulp.dest(paths.jsDest));
 });
 
-gulp.task('test:js', 'Runs unit tests', ['compile:ts'], function(done) {
+gulp.task('test:js', 'Runs unit tests', function(done) {
     new karma.Server({
         configFile: __dirname + '/karma.conf.js',
         singleRun: process.env.DEBUG ? false : true
     }, done).start();
 });
 
-gulp.task('compile:ts', 'Compile typescript for powerbi library', ['clean'], function() {
+gulp.task('compile:ts', 'Compile typescript for powerbi library', function() {
     var webpackBundle = gulp.src(['./src/powerbi.ts'])
         .pipe(webpack(webpackConfig))
         .pipe(gulp.dest('dist/'));

dist/powerbi.min.js

@@ -39,6 +39,7 @@
     "merge2": "^1.0.1",
     "phantomjs-prebuilt": "^2.1.3",
     "rimraf": "^2.5.1",
+    "run-sequence": "^1.1.5",
     "ts-loader": "^0.8.1",
     "tslint": "^3.6.0",
     "webpack": "^1.12.14",