Remove the codes relating to jshint
Toru Kobayashi committed
Jun 3, 2015 at 10:34 UTC
ce46068def53cfc9aeb957604214a562abf08dd1
15 files changed
-77
Gruntfile.js
-1
@@ -31,7 +31,6 @@ module.exports = function(grunt) {
31
'./examples/shared/*.js',
32
'.module-cache',
33
],
34
- jshint: require('./grunt/config/jshint'),
34
/*eslint-disable camelcase */
35
compare_size: require('./grunt/config/compare_size'),
36
/*eslint-enable camelcase */
grunt/config/jshint.js
deleted
-27
@@ -1,27 +0,0 @@
1
-'use strict';
2
-
3
-module.exports = {
4
- src: {
5
- options: {
6
- jshintrc: './src/.jshintrc',
7
- },
8
- files: {
9
- // We don't care about src/vendor (we shouldn't be touching this), and
10
- // tests often use JSX, which JSHint can't handle.
11
- src: [
12
- './src/**/*.js',
13
- '!./src/vendor/**',
14
- '!./src/**/__tests__/**',
15
- '!./src/test/**',
16
- ],
17
- },
18
- },
19
- project: {
20
- options: {
21
- jshintrc: './.jshintrc',
22
- },
23
- files: {
24
- src: ['./Gruntfile.js', './grunt/**/*.js', './bin/*.js', './vendor/*.js'],
25
- },
26
- },
27
-};
package.json
-1
@@ -47,7 +47,6 @@
47
"grunt-contrib-clean": "^0.6.0",
48
"grunt-contrib-compress": "^0.13.0",
49
"grunt-contrib-connect": "~0.6.0",
50
- "grunt-contrib-jshint": "^0.10.0",
50
"grunt-jest": "^0.1.2",
51
"gzip-js": "~0.3.2",
52
"jasmine-tapreporter": "~0.2.2",
src/.jshintrc
deleted
-35
@@ -1,35 +0,0 @@
1
-{
2
- "browser": true,
3
- "esnext": true,
4
-
5
- "bitwise": true,
6
- "boss": true,
7
- "curly": true,
8
- "devel": true,
9
- "eqnull": true,
10
- "expr": true,
11
- "funcscope": true,
12
- "globalstrict": true,
13
- "loopfunc": true,
14
- "newcap": false,
15
- "noempty": true,
16
- "nonstandard": true,
17
- "maxlen": 80,
18
- "sub": false,
19
- "undef": true,
20
- "unused": "vars",
21
-
22
- "globals": {
23
- "__DEV__": false,
24
- "exports": false,
25
- "module": false,
26
- "require": false,
27
-
28
- "afterEach": false,
29
- "beforeEach": false,
30
- "describe": false,
31
- "expect": false,
32
- "it": false,
33
- "spyOn": false
34
- }
35
-}
src/addons/link/__tests__/LinkedStateMixin-test.js
-1
@@ -11,7 +11,6 @@
11
12
'use strict';
13
14
-/*jshint evil:true */
14
15
describe('LinkedStateMixin', function() {
16
var LinkedStateMixin;
src/renderers/dom/client/utils/getEventModifierState.js
-1
@@ -28,7 +28,6 @@ var modifierKeyToProp = {
28
// modifier keys exposed by the event itself, does not support Lock-keys.
29
// Currently, all major browsers except Chrome seems to support Lock-keys.
30
function modifierStateGetter(keyArg) {
31
- /*jshint validthis:true */
31
var syntheticEvent = this;
32
var nativeEvent = syntheticEvent.nativeEvent;
33
if (nativeEvent.getModifierState) {
src/renderers/dom/client/wrappers/ReactDOMForm.js
-3
@@ -32,9 +32,6 @@ var ReactDOMForm = ReactClass.createClass({
32
mixins: [ReactBrowserComponentMixin, LocalEventTrapMixin],
33
34
render: function() {
35
- // TODO: Instead of using `ReactDOM` directly, we should use JSX. However,
36
- // `jshint` fails to parse JSX so in order for linting to work in the open
37
- // source repo, we need to just use `ReactDOM.form`.
35
return form(this.props);
36
},
37
src/renderers/dom/client/wrappers/ReactDOMInput.js
-1
@@ -23,7 +23,6 @@ var invariant = require('invariant');
23
var instancesByReactID = {};
24
25
function forceUpdateIfMounted() {
26
- /*jshint validthis:true */
26
if (this._rootNodeID) {
27
// DOM component is still mounted; update
28
ReactDOMInput.updateWrapper(this);
src/renderers/dom/client/wrappers/ReactDOMSelect.js
-1
@@ -29,7 +29,6 @@ var valueContextKey =
29
'__ReactDOMSelect_value$' + Math.random().toString(36).slice(2);
30
31
function updateOptionsIfPendingUpdateAndMounted() {
32
- /*jshint validthis:true */
32
if (this._pendingUpdate) {
33
this._pendingUpdate = false;
34
var value = LinkedValueUtils.getValue(this.props);
src/renderers/dom/client/wrappers/ReactDOMTextarea.js
-1
@@ -29,7 +29,6 @@ var warning = require('warning');
29
var textarea = ReactElement.createFactory('textarea');
30
31
function forceUpdateIfMounted() {
32
- /*jshint validthis:true */
32
if (this.isMounted()) {
33
this.forceUpdate();
34
}
src/renderers/dom/client/wrappers/__tests__/ReactDOMButton-test.js
-1
@@ -11,7 +11,6 @@
11
12
'use strict';
13
14
-/*jshint evil:true */
14
15
var mocks = require('mocks');
16
src/renderers/dom/client/wrappers/__tests__/ReactDOMInput-test.js
-1
@@ -11,7 +11,6 @@
11
12
'use strict';
13
14
-/*jshint evil:true */
14
15
var emptyFunction = require('emptyFunction');
16
var mocks = require('mocks');
src/renderers/dom/client/wrappers/__tests__/ReactDOMOption-test.js
-1
@@ -11,7 +11,6 @@
11
12
'use strict';
13
14
-/*jshint evil:true */
14
15
describe('ReactDOMOption', function() {
16
var React;
src/renderers/dom/client/wrappers/__tests__/ReactDOMSelect-test.js
-1
@@ -11,7 +11,6 @@
11
12
'use strict';
13
14
-/*jshint evil:true */
14
15
var mocks = require('mocks');
16
vendor/browser-transforms.js
-1
@@ -6,7 +6,6 @@
6
* LICENSE file in the root directory of this source tree. An additional grant
7
* of patent rights can be found in the PATENTS file in the same directory.
8
*/
9
-/* jshint browser: true */
9
/* jslint evil: true */
10
/*eslint-disable no-eval */
11
/*eslint-disable block-scoped-var */