Permalink
Browse files

Use .jshintrc files.

  • Loading branch information...
1 parent 33df9b7 commit a1665402511eb99e9c1a29a47262763cb3a3ba68 @scottgonzalez scottgonzalez committed Apr 20, 2012
Showing with 107 additions and 94 deletions.
  1. +12 −0 .jshintrc
  2. +20 −94 grunt.js
  3. +58 −0 tests/.jshintrc
  4. +17 −0 ui/.jshintrc
View
12 .jshintrc
@@ -0,0 +1,12 @@
+{
+ "curly": true,
+ "eqnull": true,
+ "eqeqeq": true,
+ "expr": true,
+ "latedef": true,
+ "noarg": true,
+ "onevar": true,
+ "smarttabs": true,
+ "trailing": true,
+ "undef": true
+}
View
114 grunt.js
@@ -1,3 +1,4 @@
+/*jshint node: true */
module.exports = function( grunt ) {
var // modules
@@ -293,104 +294,29 @@ grunt.initConfig({
}
},
jshint: (function() {
- var defaults = {
- curly: true,
- eqnull: true,
- eqeqeq: true,
- expr: true,
- latedef: true,
- noarg: true,
- onevar: true,
- // TODO: limit to multi-line comments https://github.com/jshint/jshint/issues/503
- smarttabs: true,
- // TODO: use "faux strict mode" https://github.com/jshint/jshint/issues/504
- // strict: true,
- trailing: true,
- undef: true
- };
+ function parserc( path ) {
+ var rc = grunt.file.readJSON( (path || "") + ".jshintrc" ),
+ settings = {
+ options: rc,
+ globals: {}
+ };
+
+ (rc.predef || []).forEach(function( prop ) {
+ settings.globals[ prop ] = true;
+ });
+ delete rc.predef;
- function extend( a, b ) {
- for ( var prop in b ) {
- a[ prop ] = b[ prop ];
- }
- return a;
+ return settings;
}
return {
- options: defaults,
- grunt: {
- options: extend({
- node: true
- }, defaults ),
- globals: {
- task: true,
- config: true,
- file: true,
- log: true,
- template: true
- }
- },
- ui: {
- options: extend({
- browser: true,
- jquery: true
- }, defaults ),
- globals: {
- Globalize: true
- }
- },
- tests: {
- options: extend({
- browser: true,
- jquery: true,
- // TODO: this is only for document.write() https://github.com/jshint/jshint/issues/519
- evil: true
- }, defaults ),
- // TODO: don't create so many globals in tests
- globals: {
- addMonths: true,
- asyncTest: true,
- container: true,
- deepEqual: true,
- d1: true,
- d2: true,
- dlg: true,
- domEqual: true,
- drag: true,
- dragged: true,
- el: true,
- equal: true,
- equalsDate: true,
- expect: true,
- Globalize: true,
- heightAfter: true,
- init: true,
- isNotOpen: true,
- isOpen: true,
- modal: true,
- module: true,
- moved: true,
- notEqual: true,
- offsetAfter: true,
- offsetBefore: true,
- ok: true,
- PROP_NAME: true,
- QUnit: true,
- restoreScroll: true,
- shouldBeDroppable: true,
- shouldmove: true,
- shouldNotBeDroppable: true,
- shouldnotmove: true,
- shouldnotresize: true,
- shouldresize: true,
- start: true,
- strictEqual: true,
- stop: true,
- test: true,
- TestHelpers: true,
- widthAfter: true
- }
- }
+ // TODO: use "faux strict mode" https://github.com/jshint/jshint/issues/504
+ // TODO: limit `smarttabs` to multi-line comments https://github.com/jshint/jshint/issues/503
+ options: parserc(),
+ ui: parserc( "ui/" ),
+ // TODO: `evil: true` is only for document.write() https://github.com/jshint/jshint/issues/519
+ // TODO: don't create so many globals in tests
+ tests: parserc( "tests/" )
};
})()
});
View
58 tests/.jshintrc
@@ -0,0 +1,58 @@
+{
+ "browser": true,
+ "curly": true,
+ "eqnull": true,
+ "eqeqeq": true,
+ "expr": true,
+ "evil": true,
+ "jquery": true,
+ "latedef": true,
+ "noarg": true,
+ "onevar": true,
+ "smarttabs": true,
+ "trailing": true,
+ "undef": true,
+ "predef": [
+ "addMonths",
+ "asyncTest",
+ "container",
+ "deepEqual",
+ "d1",
+ "d2",
+ "dlg",
+ "domEqual",
+ "drag",
+ "dragged",
+ "el",
+ "equal",
+ "equalsDate",
+ "expect",
+ "Globalize",
+ "heightAfter",
+ "init",
+ "isNotOpen",
+ "isOpen",
+ "modal",
+ "module",
+ "moved",
+ "notEqual",
+ "offsetAfter",
+ "offsetBefore",
+ "ok",
+ "PROP_NAME",
+ "QUnit",
+ "restoreScroll",
+ "shouldBeDroppable",
+ "shouldmove",
+ "shouldNotBeDroppable",
+ "shouldnotmove",
+ "shouldnotresize",
+ "shouldresize",
+ "start",
+ "strictEqual",
+ "stop",
+ "test",
+ "TestHelpers",
+ "widthAfter"
+ ]
+}
View
17 ui/.jshintrc
@@ -0,0 +1,17 @@
+{
+ "browser": true,
+ "curly": true,
+ "eqnull": true,
+ "eqeqeq": true,
+ "expr": true,
+ "jquery": true,
+ "latedef": true,
+ "noarg": true,
+ "onevar": true,
+ "smarttabs": true,
+ "trailing": true,
+ "undef": true,
+ "predef": [
+ "Globalize"
+ ]
+}

0 comments on commit a166540

Please sign in to comment.