Build: Enable ESLint one-var rule for var declarations in browser code

Node.js code is written more & more commonly in ES6+ so it doesn't make sense
to enable it there. There are many violations in test code so it's disabled
there as well.

Closes gh-4615
This commit is contained in:
Michał Gołębiowski-Owczarek 2020-03-02 22:25:35 +01:00 committed by GitHub
parent 364476c3dc
commit 4a7fc8544e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 10 additions and 6 deletions

View File

@ -19,6 +19,7 @@
},
"rules": {
"one-var": ["error", {"var": "always"}],
"strict": ["error", "function"]
}
}

3
dist/.eslintrc.json vendored
View File

@ -19,7 +19,8 @@
"rules": {
// That is okay for the built version
"no-multiple-empty-lines": "off"
"no-multiple-empty-lines": "off",
"one-var": "off"
}
}
]

View File

@ -8,7 +8,7 @@ var reliableTrDimensionsVal;
// IE/Edge misreport `getComputedStyle` of table rows with width/height
// set in CSS while `offset*` properties report correct values.
support.reliableTrDimensions = function() {
var table, tr, trChild;
var table, tr, trChild, trStyle;
if ( reliableTrDimensionsVal == null ) {
table = document.createElement( "table" );
tr = document.createElement( "tr" );
@ -23,7 +23,7 @@ support.reliableTrDimensions = function() {
.appendChild( tr )
.appendChild( trChild );
var trStyle = window.getComputedStyle( tr );
trStyle = window.getComputedStyle( tr );
reliableTrDimensionsVal = parseInt( trStyle.height ) > 3;
documentElement.removeChild( table );

View File

@ -288,11 +288,12 @@ jQuery.event = {
dispatch: function( nativeEvent ) {
// Make a writable jQuery.Event from the native event object
var event = jQuery.event.fix( nativeEvent );
var i, j, ret, matched, handleObj, handlerQueue,
args = new Array( arguments.length ),
// Make a writable jQuery.Event from the native event object
event = jQuery.event.fix( nativeEvent ),
handlers = ( dataPriv.get( this, "events" ) || {} )[ event.type ] || [],
special = jQuery.event.special[ event.type ] || {};

View File

@ -47,6 +47,7 @@
"brace-style": "off",
"key-spacing": "off",
"camelcase": "off",
"one-var": "off",
"strict": "off",
// Not really too many - waiting for autofix features for these rules