Merge pull request #818 from fewieden/fix-typo

fixed typo + eslint
This commit is contained in:
Michael Teeuw 2017-04-03 20:21:35 +02:00 committed by GitHub
commit 22e4e4125a
5 changed files with 51 additions and 21 deletions

View File

@ -6,12 +6,20 @@ module.exports = function(grunt) {
options: { options: {
configFile: ".eslintrc.json" configFile: ".eslintrc.json"
}, },
target: ["js/*.js", "modules/default/*.js", "modules/default/*/*.js", target: [
"serveronly/*.js", "*.js", "tests/*/*.js", "!modules/default/alert/notificationFx.js", "js/*.js",
"!modules/default/alert/modernizr.custom.js", "!modules/default/alert/classie.js", "modules/default/*.js",
"modules/default/*/*.js",
"serveronly/*.js",
"*.js",
"tests/**/*.js",
"!modules/default/alert/notificationFx.js",
"!modules/default/alert/modernizr.custom.js",
"!modules/default/alert/classie.js",
"config/*", "config/*",
"translations/translations.js", "vendor/vendor.js" "translations/translations.js",
"vendor/vendor.js",
"modules/node_modules/node_helper/index.js"
] ]
}, },
stylelint: { stylelint: {
@ -19,14 +27,26 @@ module.exports = function(grunt) {
options: { options: {
configFile: ".stylelintrc" configFile: ".stylelintrc"
}, },
src: ["css/main.css", "modules/default/calendar/calendar.css", "modules/default/clock/clock_styles.css", "modules/default/currentweather/currentweather.css", "modules/default/weatherforcast/weatherforcast.css"] src: [
"css/main.css",
"modules/default/calendar/calendar.css",
"modules/default/clock/clock_styles.css",
"modules/default/currentweather/currentweather.css",
"modules/default/weatherforcast/weatherforcast.css"
]
} }
}, },
jsonlint: { jsonlint: {
main: { main: {
src: ["package.json", ".eslintrc.json", ".stylelintrc", "translations/*.json", src: [
"modules/default/*/translations/*.json", "installers/pm2_MagicMirror.json", "package.json",
"vendor/package.js"], ".eslintrc.json",
".stylelintrc",
"translations/*.json",
"modules/default/*/translations/*.json",
"installers/pm2_MagicMirror.json",
"vendor/package.js"
],
options: { options: {
reporter: "jshint" reporter: "jshint"
} }
@ -58,11 +78,20 @@ module.exports = function(grunt) {
"MD038": false "MD038": false
} }
}, },
src: ["README.md", "CHANGELOG.md", "LICENSE.md", "modules/README.md", "modules/default/**/*.md", "!modules/default/calendar/vendor/ical.js/readme.md"] src: [
"README.md",
"CHANGELOG.md",
"LICENSE.md",
"modules/README.md",
"modules/default/**/*.md",
"!modules/default/calendar/vendor/ical.js/readme.md"
]
} }
}, },
yamllint: { yamllint: {
all: [".travis.yml"] all: [
".travis.yml"
]
} }
}); });
grunt.loadNpmTasks("grunt-eslint"); grunt.loadNpmTasks("grunt-eslint");

View File

@ -20,7 +20,7 @@ NodeHelper = Class.extend({
}, },
start: function() { start: function() {
console.log("Staring module helper: " + this.name); console.log("Starting module helper: " + this.name);
}, },
/* socketNotificationReceived(notification, payload) /* socketNotificationReceived(notification, payload)
@ -45,7 +45,7 @@ NodeHelper = Class.extend({
/* setPath(path) /* setPath(path)
* Set the module path. * Set the module path.
* *
* argument name string - Module name. * argument path string - Module path.
*/ */
setPath: function(path) { setPath: function(path) {
this.path = path; this.path = path;
@ -98,9 +98,10 @@ NodeHelper = Class.extend({
// register catch all. // register catch all.
socket.on("*", function(notification, payload) { socket.on("*", function(notification, payload) {
if (notification !== "*") if (notification !== "*") {
//console.log('received message in namespace: ' + namespace); //console.log('received message in namespace: ' + namespace);
self.socketNotificationReceived(notification, payload); self.socketNotificationReceived(notification, payload);
}
}); });
}); });

View File

@ -1,4 +1,4 @@
/* Magic Mirror Test config for default compliments /* Magic Mirror Test config for default compliments
* *
* By Rodrigo Ramírez Norambuena https://rodrigoramirez.com * By Rodrigo Ramírez Norambuena https://rodrigoramirez.com
* MIT Licensed. * MIT Licensed.

View File

@ -24,8 +24,8 @@ var config = {
// Using exotic content. This is why dont accept go to JSON configuration file // Using exotic content. This is why dont accept go to JSON configuration file
(function() { (function() {
var positions = ["top_bar", "top_left", "top_center", "top_right", "upper_third", var positions = ["top_bar", "top_left", "top_center", "top_right", "upper_third",
"middle_center", "lower_third", "bottom_left", "bottom_center", "bottom_right", "middle_center", "lower_third", "bottom_left", "bottom_center", "bottom_right",
"bottom_bar", "fullscreen_above", "fullscreen_below"]; "bottom_bar", "fullscreen_above", "fullscreen_below"];
var modules = Array(); var modules = Array();
for (idx in positions) { for (idx in positions) {
modules.push({ modules.push({

View File

@ -63,8 +63,8 @@ describe("'global.root_path' set in js/app.js", function() {
}); });
it("should expect the global.version equals package.json file", function() { it("should expect the global.version equals package.json file", function() {
version_package = JSON.parse(fs.readFileSync("package.json", "utf8")).version; versionPackage = JSON.parse(fs.readFileSync("package.json", "utf8")).version;
expect(this.sandbox.global.version).to.equal(version_package); expect(this.sandbox.global.version).to.equal(versionPackage);
}); });
}); });