Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix JSHint build #1052

Closed
wants to merge 1 commit into from
Closed

Fix JSHint build #1052

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion grunt/config/webdriver-all.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,4 +110,4 @@ module.exports = function(props){
}

return exports;
}
};
2 changes: 1 addition & 1 deletion grunt/config/webdriver-perf.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ var params = []
)
.concat(reactVersions
.map(encodeURIComponent)
.map(function(version){ return 'react=' + version }
.map(function(version){ return 'react=' + version; }
)
);

Expand Down
4 changes: 2 additions & 2 deletions grunt/tasks/npm-react-tools.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
var fs = require('fs');
var grunt = require('grunt');

function pack() {
var pack = function() {
var done = this.async();
var spawnCmd = {
cmd: 'npm',
Expand All @@ -14,7 +14,7 @@ function pack() {
var dest = 'build/react-tools.tgz';
fs.rename(src, dest, done);
});
}
};

module.exports = {
pack: pack
Expand Down
6 changes: 3 additions & 3 deletions grunt/tasks/npm-react.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ function buildRelease() {
grunt.file.write(dest + 'package.json', JSON.stringify(pkg, null, 2));
}

function packRelease() {
var packRelease = function() {
var done = this.async();
var spawnCmd = {
cmd: 'npm',
Expand All @@ -47,11 +47,11 @@ function packRelease() {
}
};
grunt.util.spawn(spawnCmd, function() {
var src = 'build/react-' + grunt.config.data.pkg.version + '.tgz'
var src = 'build/react-' + grunt.config.data.pkg.version + '.tgz';
var dest = 'build/react.tgz';
fs.rename(src, dest, done);
});
}
};

module.exports = {
buildRelease: buildRelease,
Expand Down
6 changes: 3 additions & 3 deletions src/browser/ReactEventEmitter.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* @providesModule ReactEventEmitter
* @typechecks static-only
*/

/* jshint maxlen: 90 */
"use strict";

var EventConstants = require('EventConstants');
Expand Down Expand Up @@ -227,7 +227,7 @@ var ReactEventEmitter = merge(ReactEventEmitterMixin, {
* top-level listeners to the document object only, at least for these
* movement types of events and possibly all events.
*
* @see http://www.quirksmode.org/blog/archives/2010/09/click_event_del.html
* @see http://quirksmode.org/blog/archives/2010/09/click_event_del.html
*
* Also, `keyup`/`keypress`/`keydown` do not bubble to the window on IE, but
* they bubble to document.
Expand Down Expand Up @@ -275,7 +275,7 @@ var ReactEventEmitter = merge(ReactEventEmitterMixin, {
trapCapturedEvent(topLevelTypes.topBlur, 'blur', mountAt);
} else if (isEventSupported('focusin')) {
// IE has `focusin` and `focusout` events which bubble.
// @see http://www.quirksmode.org/blog/archives/2008/04/delegating_the.html
// @see http://quirksmode.org/blog/archives/2008/04/delegating_the.html
trapBubbledEvent(topLevelTypes.topFocus, 'focusin', mountAt);
trapBubbledEvent(topLevelTypes.topBlur, 'focusout', mountAt);
}
Expand Down
3 changes: 2 additions & 1 deletion src/utils/isTextInputElement.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,12 @@
*
* @providesModule isTextInputElement
*/
/* jshint maxlen: 120 */

"use strict";

/**
* @see http://www.whatwg.org/specs/web-apps/current-work/multipage/the-input-element.html#input-type-attr-summary
* @see http://whatwg.org/specs/web-apps/current-work/multipage/the-input-element.html#input-type-attr-summary
*/
var supportedInputTypes = {
'color': true,
Expand Down