Merge pull request #21 from walaura/dev

eslinted everything

.eslintrc.json

@@ -0,0 +1,32 @@
+{
+    "env": {
+        "browser": true,
+        "commonjs": true,
+		"mocha": true,
+		"phantomjs": true,
+        "es6": true
+    },
+    "extends": "eslint:recommended",
+    "parserOptions": {
+        "sourceType": "module"
+    },
+    "rules": {
+        "indent": [
+            "error",
+            "tab"
+        ],
+        "linebreak-style": [
+            "error",
+            "unix"
+        ],
+        "quotes": [
+            "error",
+            "single"
+        ],
+        "semi": [
+            "error",
+            "always"
+        ],
+        "no-console": 0
+    }
+}

gulpfile.js

@@ -5,7 +5,6 @@ const uglify = require('gulp-uglify');
 const header = require('gulp-header');
 const path = require('path');
 const fs = require('fs-extra');
-const WrapperPlugin = require('wrapper-webpack-plugin');
 
 const config = require('./src/conf.js');
 
@@ -39,7 +38,7 @@ gulp.task('clean', () => {
 	['dist','temp','temp/screenshots'].map((dir)=>{
 		fs.removeSync(dir);
 		fs.mkdirSync(dir);
-	})
+	});
 });
 
 
@@ -75,7 +74,7 @@ gulp.task('test', ['make'], function () {
 			if(uploadCount >= screenshots.length) {
 				cb(null, chunk);
 			}
-		}
+		};
 
 		screenshots.map((screenshot)=>{
 
@@ -87,13 +86,13 @@ gulp.task('test', ['make'], function () {
 			]);
 
 			child.stdout.on('data', (buffer) => {
-				gutil.log('Look at it!!!', gutil.colors.magenta(buffer.toString().replace("\n",'')));
+				gutil.log('Look at it!!!', gutil.colors.magenta(buffer.toString().replace('\n','')));
 			});
 			child.stdout.on('end', isDoneMaybe);
 
 		});
 
-	}))
+	}));
 });
 
 
@@ -113,7 +112,7 @@ gulp.task('default', function() {
 				root: path.resolve('./src')
 			}
 		}))
-		.pipe(header(config.webpack.header+"\n"))
+		.pipe(header(config.webpack.header+'\n'))
 		.pipe(gulp.dest('dist/'));
 });
 
@@ -133,9 +132,9 @@ gulp.task('make', ['clean'], function() {
 			}
 		}))
 		.pipe(uglify())
-		.pipe(header(config.webpack.header+"\n"))
-		.pipe(gulp.dest('dist/'))
-})
+		.pipe(header(config.webpack.header+'\n'))
+		.pipe(gulp.dest('dist/'));
+});
 
 
 gulp.task('release', function(){
@@ -146,7 +145,8 @@ gulp.task('release', function(){
 		.pipe(release({
 			manifest: require('./package.json')
 		}).on('error',function(e){
+			console.error(e);
 			this.emit('end');
-		}))
+		}));
 
 });

package.json

@@ -28,6 +28,7 @@
     "wrapper-webpack-plugin": "0.1.11"
   },
   "devDependencies": {
+    "eslint": "^3.16.1",
     "gulp-github-release": "^1.2.0",
     "mocha": "^3.2.0",
     "mocha-phantomjs": "^4.1.0",
@@ -49,6 +50,7 @@
   "scripts": {
     "gulp": "./node_modules/.bin/gulp",
     "install": "gulp make",
+    "eslint": "eslint --ignore-path .gitignore .",
     "release": "npm publish && npm install && gulp release",
     "test": "gulp test"
   }

src/app.js

@@ -19,7 +19,7 @@ const constellation = function ({
 } = {}) {
 
 
-	if(padding[0] === 0 && padding[1] === 0) padding = [fuzziness,fuzziness]
+	if(padding[0] === 0 && padding[1] === 0) padding = [fuzziness,fuzziness];
 	style = Object.assign({}, config.defaults.style, style);
 	speed = Object.assign({}, config.defaults.speed, speed);
 
@@ -40,7 +40,7 @@ const constellation = function ({
 	);
 
 
-	const onDOMReady = (resolve,reject) => {
+	const onDOMReady = (resolve) => {
 
 		let isJiggling = true;
 		let pointerPosition = [0,0];
@@ -62,7 +62,7 @@ const constellation = function ({
 			isJiggling = false;
 			pointerPosition = [x,y];
 		});
-		canvas.addEventListener('mouseout',(ev)=>{
+		canvas.addEventListener('mouseout',()=>{
 			isJiggling = true;
 		});
 
@@ -79,10 +79,10 @@ const constellation = function ({
 					pointerPosition: pointerPosition,
 					isJiggling: isJiggling
 				}
-			)
-		}
+			);
+		};
 
-		puppis.addEventListener('message', (msg)=>{
+		puppis.addEventListener('message', msg => {
 			text.is(
 				msg,'updateComplete',
 				(payload) => {
@@ -92,9 +92,9 @@ const constellation = function ({
 							lines: payload.lines
 						});
 						repaint();
-					})
+					});
 				}
-			)
+			);
 		});
 
 		repaint();

src/class/Canvas.js

@@ -24,16 +24,16 @@ export default class Canvas {
 		ctx.strokeStyle = style.lineColor;
 		ctx.globalCompositeOperation = 'source-over';
 		for (let i = 0, len = objects.lines.length; i < len; i++) {
-			let line = objects.lines[i]
+			let line = objects.lines[i];
 			if(onDraw.line) {
 				onDraw.line(ctx,style,line);
 			}
 			else {
 				ctx.beginPath();
-					ctx.moveTo(line.pos[0],line.pos[1]);
-					ctx.lineTo(line.pos[2],line.pos[3]);
-					ctx.stroke();
-					ctx.closePath();
+				ctx.moveTo(line.pos[0],line.pos[1]);
+				ctx.lineTo(line.pos[2],line.pos[3]);
+				ctx.stroke();
+				ctx.closePath();
 			}
 			if(onDraw.afterLine) onDraw.afterLine(ctx,style,line);
 		}
@@ -43,13 +43,13 @@ export default class Canvas {
 			ctx.fillStyle = '#f0f';
 			ctx.globalCompositeOperation = 'destination-out';
 			for (let i = 0, len = objects.stars.length; i < len; i++) {
-				let star = objects.stars[i]
+				let star = objects.stars[i];
 				ctx.beginPath();
-					ctx.arc(
+				ctx.arc(
 						star.pos[0], star.pos[1],
 						(style.starSize + style.starPadding),
 						0, 2 * Math.PI);
-					ctx.fill();
+				ctx.fill();
 				ctx.closePath();
 			}
 		}
@@ -58,20 +58,20 @@ export default class Canvas {
 		ctx.fillStyle = style.starColor;
 		ctx.globalCompositeOperation = 'source-over';
 		for (let i = 0, len = objects.stars.length; i < len; i++) {
-			let star = objects.stars[i]
+			let star = objects.stars[i];
 			if(onDraw.star) {
 				onDraw.star(ctx,style,star);
 			}
 			else {
 				ctx.beginPath();
-					ctx.arc(
+				ctx.arc(
 						star.pos[0], star.pos[1], style.starSize,0, 2 * Math.PI
 					);
-					ctx.fill();
+				ctx.fill();
 				ctx.closePath();
 			}
 			if(onDraw.afterStar) onDraw.afterStar(ctx,style,star);
-		};
+		}
 		ctx.closePath();
 
 		if(onDraw.afterFrame) onDraw.afterFrame(ctx,style,objects);

src/conf.js

@@ -28,4 +28,4 @@ module.exports = {
 			passive: .075
 		}
 	}
-}
+};

src/lib/random.js

@@ -1,3 +1,3 @@
 export default (arr) => {
 	return arr[Math.floor(Math.random() * arr.length)];
-}
+};

src/lib/text.js

@@ -7,7 +7,7 @@ export default {
 	},
 	is: (msg,body,cb) => {
 		if(msg.data.body === body) {
-			cb(msg.data.payload)
+			cb(msg.data.payload);
 		}
 	}
-}
+};