Merge pull request #3416 from koba04/remove-jshint

Remove the codes relating to jshint

Gruntfile.js

@@ -32,7 +32,6 @@ module.exports = function(grunt) {
       './examples/shared/*.js',
       '.module-cache',
     ],
-    jshint: require('./grunt/config/jshint'),
     /*eslint-disable camelcase */
     compare_size: require('./grunt/config/compare_size'),
     /*eslint-enable camelcase */

grunt/config/jshint.js

@@ -1,27 +0,0 @@
-'use strict';
-
-module.exports = {
-  src: {
-    options: {
-      jshintrc: './src/.jshintrc',
-    },
-    files: {
-      // We don't care about src/vendor (we shouldn't be touching this), and
-      // tests often use JSX, which JSHint can't handle.
-      src: [
-        './src/**/*.js',
-        '!./src/vendor/**',
-        '!./src/**/__tests__/**',
-        '!./src/test/**',
-      ],
-    },
-  },
-  project: {
-    options: {
-      jshintrc: './.jshintrc',
-    },
-    files: {
-      src: ['./Gruntfile.js', './grunt/**/*.js', './bin/*.js', './vendor/*.js'],
-    },
-  },
-};

package.json

@@ -48,7 +48,6 @@
     "grunt-contrib-clean": "^0.6.0",
     "grunt-contrib-compress": "^0.13.0",
     "grunt-contrib-connect": "~0.6.0",
-    "grunt-contrib-jshint": "^0.10.0",
     "grunt-jest": "^0.1.2",
     "gzip-js": "~0.3.2",
     "jasmine-tapreporter": "~0.2.2",

src/.jshintrc

@@ -1,35 +0,0 @@
-{
-  "browser": true,
-  "esnext": true,
-
-  "bitwise": true,
-  "boss": true,
-  "curly": true,
-  "devel": true,
-  "eqnull": true,
-  "expr": true,
-  "funcscope": true,
-  "globalstrict": true,
-  "loopfunc": true,
-  "newcap": false,
-  "noempty": true,
-  "nonstandard": true,
-  "maxlen": 80,
-  "sub": false,
-  "undef": true,
-  "unused": "vars",
-
-  "globals": {
-    "__DEV__": false,
-    "exports": false,
-    "module": false,
-    "require": false,
-
-    "afterEach": false,
-    "beforeEach": false,
-    "describe": false,
-    "expect": false,
-    "it": false,
-    "spyOn": false
-  }
-}

src/addons/link/__tests__/LinkedStateMixin-test.js

@@ -11,7 +11,6 @@
 
 'use strict';
 
-/*jshint evil:true */
 
 describe('LinkedStateMixin', function() {
   var LinkedStateMixin;

src/renderers/dom/client/utils/getEventModifierState.js

@@ -28,7 +28,6 @@ var modifierKeyToProp = {
 // modifier keys exposed by the event itself, does not support Lock-keys.
 // Currently, all major browsers except Chrome seems to support Lock-keys.
 function modifierStateGetter(keyArg) {
-  /*jshint validthis:true */
   var syntheticEvent = this;
   var nativeEvent = syntheticEvent.nativeEvent;
   if (nativeEvent.getModifierState) {

src/renderers/dom/client/wrappers/ReactDOMForm.js

@@ -32,9 +32,6 @@ var ReactDOMForm = ReactClass.createClass({
   mixins: [ReactBrowserComponentMixin, LocalEventTrapMixin],
 
   render: function() {
-    // TODO: Instead of using `ReactDOM` directly, we should use JSX. However,
-    // `jshint` fails to parse JSX so in order for linting to work in the open
-    // source repo, we need to just use `ReactDOM.form`.
     return form(this.props);
   },
 

src/renderers/dom/client/wrappers/ReactDOMInput.js

@@ -23,7 +23,6 @@ var invariant = require('invariant');
 var instancesByReactID = {};
 
 function forceUpdateIfMounted() {
-  /*jshint validthis:true */
   if (this._rootNodeID) {
     // DOM component is still mounted; update
     ReactDOMInput.updateWrapper(this);

src/renderers/dom/client/wrappers/ReactDOMSelect.js

@@ -29,7 +29,6 @@ var valueContextKey =
   '__ReactDOMSelect_value$' + Math.random().toString(36).slice(2);
 
 function updateOptionsIfPendingUpdateAndMounted() {
-  /*jshint validthis:true */
   if (this._pendingUpdate) {
     this._pendingUpdate = false;
     var value = LinkedValueUtils.getValue(this.props);

src/renderers/dom/client/wrappers/ReactDOMTextarea.js

@@ -29,7 +29,6 @@ var warning = require('warning');
 var textarea = ReactElement.createFactory('textarea');
 
 function forceUpdateIfMounted() {
-  /*jshint validthis:true */
   if (this.isMounted()) {
     this.forceUpdate();
   }

src/renderers/dom/client/wrappers/__tests__/ReactDOMButton-test.js

@@ -11,7 +11,6 @@
 
 'use strict';
 
-/*jshint evil:true */
 
 var mocks = require('mocks');
 

src/renderers/dom/client/wrappers/__tests__/ReactDOMInput-test.js

@@ -11,7 +11,6 @@
 
 'use strict';
 
-/*jshint evil:true */
 
 var emptyFunction = require('emptyFunction');
 var mocks = require('mocks');

src/renderers/dom/client/wrappers/__tests__/ReactDOMOption-test.js

@@ -11,7 +11,6 @@
 
 'use strict';
 
-/*jshint evil:true */
 
 describe('ReactDOMOption', function() {
   var React;

src/renderers/dom/client/wrappers/__tests__/ReactDOMSelect-test.js

@@ -11,7 +11,6 @@
 
 'use strict';
 
-/*jshint evil:true */
 
 var mocks = require('mocks');
 

vendor/browser-transforms.js

@@ -6,7 +6,6 @@
  * LICENSE file in the root directory of this source tree. An additional grant
  * of patent rights can be found in the PATENTS file in the same directory.
  */
-/* jshint browser: true */
 /* jslint evil: true */
 /*eslint-disable no-eval */
 /*eslint-disable block-scoped-var */