Made build workflow better

Author: leo

gulpfile.js

@@ -3,11 +3,12 @@ const gulp = require('gulp')
 const del = require('del')
 const babel = require('gulp-babel')
 
-gulp.task('compile', () => {
-  return gulp.src('src/**/*.js')
+const path = 'src/**/*.js'
+
+gulp.task('default', () => {
+  return gulp.src(path)
   .pipe(babel())
   .pipe(gulp.dest('dist'))
 })
 
 gulp.task('clean', () => del(['dist']))
-gulp.task('default', ['compile'])

package.json

@@ -8,7 +8,8 @@
     "dist"
   ],
   "scripts": {
-    "gulp": "gulp",
+    "build": "gulp",
+    "prepublish": "npm run build",
     "test": "xo && ava"
   },
   "babel": {
@@ -53,7 +54,11 @@
     "envs": [
       "browser",
       "jsx"
-    ]
+    ],
+    "rules": {
+      "react/require-default-props": 0,
+      "react/no-unused-prop-types": 0
+    }
   },
   "devDependencies": {
     "ava": "0.22.0",

src/frame.js

@@ -1,11 +1,12 @@
+// Packages
 import React from 'react'
 import PropTypes from 'prop-types'
 
 export default class Frame extends React.Component {
   static propTypes = {
     children: PropTypes.any,
     component: PropTypes.any,
-    duration: PropTypes.number, // eslint-disable-line react/no-unused-prop-types
+    duration: PropTypes.number,
     onRender: PropTypes.func
   };
 
@@ -25,12 +26,14 @@ export default class Frame extends React.Component {
   render() {
     const {component} = this.props
     const props = {}
+
     Object.keys(this.props).forEach(k => {
       if (Frame.propTypes[k]) {
         return
       }
       props[k] = this.props[k]
     })
+
     return React.createElement(component, props, this.props.children)
   }
 }

src/index.js

@@ -1,2 +1,3 @@
+// Utilities
 export {default as Keyframes} from './keyframes'
 export {default as Frame} from './frame'

src/keyframes.js

@@ -1,5 +1,8 @@
+// Packages
 import React from 'react'
 import PropTypes from 'prop-types'
+
+// Utilities
 import Frame from './frame'
 
 const noop = () => {}
@@ -10,9 +13,9 @@ export default class Keyframes extends React.Component {
     component: PropTypes.any, // eslint-disable-line react/no-unused-prop-types
     delay: PropTypes.number,
     loop: PropTypes.oneOfType([
-      React.PropTypes.string,
-      React.PropTypes.number,
-      React.PropTypes.bool
+      PropTypes.string,
+      PropTypes.number,
+      PropTypes.bool
     ]),
     onStart: PropTypes.func,
     onEnd: PropTypes.func
@@ -75,7 +78,7 @@ export default class Keyframes extends React.Component {
 
     const props = {}
     Object.keys(this.props).forEach(k => {
-      // don't pass props which exist only on Keyframes
+      // Don't pass props which exist only on Keyframes
       if (Keyframes.propTypes[k] && !Frame.propTypes[k]) {
         return
       }