2 commits

Author	SHA1	Message	Date
Joshua Spence	fcaeb2aeb6	Be more strict with JSHint. ... Summary: Add a bunch of extra checks to be performed by `jshint`. For documentation, see http://jshint.com/docs/options/. Test Plan: Ran `jshint --config support/jshint/jshintconfig webroot/rsrc/js/`. There were a bunch of existing violations, but some of these are legitimate and probably require attention. ```lang=json { "bitwise": true, // 0 violations "curly": true, // 0 violations "immed": true, // 1 violation "indent": 2, // 0 violations "latedef": true, // 10 violations "newcap": true, // 1 violation "noarg": true, // 0 violations "quotmark": "single", // 55 violations "undef": true, // 24 violations "unused": true, // 107 violations "expr": true, "loopfunc": true, "sub": true, "globals": { "JX": false, "__DEV__": false }, "browser": true } ``` Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D9659	2014-06-23 04:07:52 +10:00
epriestley	0569218201	Use JsShrink if jsxmin is not available ... Summary: If `jsxmin` is not available, use a pure PHP implementation instead (JsShrink). Test Plan: - Ran `arc lint --lintall` on all JS and fixed every relevant warning. - Forced minification on and browsed around the site using JS behaviors. Didn't hit anything problematic. Reviewers: vrana, btrahan Reviewed By: vrana CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5670	2013-05-18 17:04:22 -07:00