canvas-lms/.jshintrc

{
  "browser": true,
  "onevar": false,
  "jquery": true,
  "asi": true,
  "plusplus": false,
  "curly": true,
  "forin": true,
  "strict": false,
  "undef": true
}
add a rake canvas:check_syntax task to check for js errors you can pass quick=true to to disable style guideline checks and only check for syntax errors. this will also spit out javascript style warnings using jshint and gjslint. I have not nailed down exactly which things we want to warn about, we can talk about it and come to a consensus on what we care about. I do not necessarily think that we should use both jshint and gjslint, I just put them both in to show what kind of things they can check for. in order to use gjslint, you need to install it from http://code.google.com/closure/utilities/ Make your .git/hooks/pre-commit look like: rake canvas:check_syntax:changed quick=true to not allow committing js with syntax errors Change-Id: Icfc21c2d30f60dc2fafef80d5c93432b6309afdb Reviewed-on: https://gerrit.instructure.com/2908 Tested-by: Hudson <hudson@instructure.com> Reviewed-by: Zach Wily <zach@instructure.com> 2011-03-31 05:06:00 +08:00			`{`
			`"browser": true,`
			`"onevar": false,`
			`"jquery": true,`
			`"asi": true,`
			`"plusplus": false,`
			`"curly": true,`
			`"forin": true,`
			`"strict": false,`
			`"undef": true`
			`}`