From 9ec55879a1e2e7c19475d93d0353deb940b984da Mon Sep 17 00:00:00 2001
From: Gary Katsevman <git@gkatsev.com>
Date: Tue, 3 Jan 2017 15:05:58 -0500
Subject: [PATCH] chore: better dev experience (#3896)

npm start now runs `grunt dev` which in turn runs the connect server,
sets up babel watching, browserify watching (for both src and test
files), and css and alternate builds watching.

Additionally, it will copy over the dev build files into the dist folder
so that if you're linking videojs and you have a project that gets
video.js from the dist, you can link and develop with more easy without
rerunning all of `grunt dist` each time.

Additionally, this specifically does not run more time consumming
operations like minification.
---
 build/grunt.js | 29 ++++++++++++++++++++++++-----
 package.json   |  5 +++--
 2 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/build/grunt.js b/build/grunt.js
index 8b396af03..e89286e18 100644
--- a/build/grunt.js
+++ b/build/grunt.js
@@ -127,6 +127,15 @@ module.exports = function(grunt) {
     },
     dist: {},
     watch: {
+      dist: {
+        files: [
+          'build/temp/video.js',
+          'build/temp/alt/video.novtt.js',
+          'build/temp/video-js.css',
+          'build/temp/alt/video-js-cdn.css'
+        ],
+        tasks: ['copy:dist']
+      },
       novtt: {
         files: ['build/temp/video.js'],
         tasks: ['concat:novtt']
@@ -142,10 +151,6 @@ module.exports = function(grunt) {
       babel: {
         files: ['src/js/**/*.js'],
         tasks: ['babel:es5']
-      },
-      jshint: {
-        files: ['src/**/*', 'test/unit/**/*.js', 'Gruntfile.js'],
-        tasks: 'jshint'
       }
     },
     connect: {
@@ -405,6 +410,14 @@ module.exports = function(grunt) {
         'watch:babel',
         'browserify:tests'
       ],
+      dev: [
+        'browserify:watch',
+        'browserify:tests',
+        'watch:novtt',
+        'watch:skin',
+        'watch:dist',
+        'shell:babel'
+      ],
       // Run multiple watch tasks in parallel
       // Needed so watchify can cache intelligently
       watchAll: [
@@ -437,6 +450,12 @@ module.exports = function(grunt) {
       }
     },
     shell: {
+      babel: {
+        command: 'npm run babel -- --w',
+        options: {
+          preferLocal: true
+        }
+      },
       lint: {
         command: 'npm run lint',
         options: {
@@ -540,7 +559,7 @@ module.exports = function(grunt) {
     'test-a11y'].concat(process.env.TRAVIS && 'coveralls').filter(Boolean));
 
   // Run while developing
-  grunt.registerTask('dev', ['build', 'connect:dev', 'concurrent:watchSandbox']);
+  grunt.registerTask('dev', ['connect:dev', 'concurrent:dev']);
 
   grunt.registerTask('watchAll', ['build', 'connect:dev', 'concurrent:watchAll']);
 
diff --git a/package.json b/package.json
index d57af7f57..ab114a92c 100644
--- a/package.json
+++ b/package.json
@@ -22,12 +22,13 @@
     "clean": "grunt clean",
     "grunt": "grunt",
     "lint": "vjsstandard",
-    "start": "grunt watchAll",
+    "start": "grunt dev",
     "test": "grunt test",
     "docs": "npm run docs:lint && npm run docs:api",
     "docs:api": "jsdoc -r src/js -d docs/api -c .jsdoc.json",
     "docs:lint": "remark -- './**/*.md'",
-    "docs:fix": "remark --output -- './**/*.md'"
+    "docs:fix": "remark --output -- './**/*.md'",
+    "babel": "babel src/js -d es5"
   },
   "repository": {
     "type": "git",