jquery - npm Package Compare versions

Comparing version 3.0.0 to 3.1.0

dist/core.js

@@ -0,1 +1,5 @@
+/* global Symbol */
+// Defining this global in .eslintrc would create a danger of using the global
+// unguarded in another place, it seems safer to define global only for this module
+
 define( [
@@ -23,3 +27,3 @@ "./var/arr",
 var
-	version = "3.0.0",
+	version = "3.1.0",
 
@@ -256,3 +260,7 @@ // Define a local copy of jQuery
 	isEmptyObject: function( obj ) {
+
+		/* eslint-disable no-unused-vars */
+		// See https://github.com/eslint/eslint/issues/6125
 		var name;
+
 		for ( name in obj ) {
@@ -447,11 +455,5 @@ return false;
 
-// JSHint would error on this code due to the Symbol not being defined in ES5.
-// Defining this global in .jshintrc would create a danger of using the global
-// unguarded in another place, it seems safer to just disable JSHint for these
-// three lines.
-/* jshint ignore: start */
 if ( typeof Symbol === "function" ) {
 	jQuery.fn[ Symbol.iterator ] = arr[ Symbol.iterator ];
 }
-/* jshint ignore: end */
 
@@ -458,0 +460,0 @@ // Populate the class2type map

package.json

@@ -5,3 +5,3 @@ {
   "description": "JavaScript library for DOM operations",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "main": "dist/jquery.js",
@@ -11,3 +11,3 @@ "homepage": "https://jquery.com",
     "name": "jQuery Foundation and other contributors",
-    "url": "https://github.com/jquery/jquery/blob/3.0.0/AUTHORS.txt"
+    "url": "https://github.com/jquery/jquery/blob/3.1.0/AUTHORS.txt"
   },
@@ -34,2 +34,3 @@ "repository": {
     "cross-spawn": "2.2.3",
+    "eslint-config-jquery": "0.1.6",
     "grunt": "1.0.1",
@@ -39,7 +40,6 @@ "grunt-babel": "6.0.0",
     "grunt-compare-size": "0.4.2",
-    "grunt-contrib-jshint": "1.0.0",
     "grunt-contrib-uglify": "1.0.1",
     "grunt-contrib-watch": "1.0.0",
+    "grunt-eslint": "18.1.0",
     "grunt-git-authors": "3.2.0",
-    "grunt-jscs": "2.8.0",
     "grunt-jsonlint": "1.0.7",
@@ -46,0 +46,0 @@ "grunt-newer": "1.2.0",

src/ajax.js

@@ -309,2 +309,3 @@ define( [
 		contentType: "application/x-www-form-urlencoded; charset=UTF-8",
+
 		/*
@@ -311,0 +312,0 @@ timeout: 0,

src/attributes/val.js

@@ -146,2 +146,5 @@ define( [
 					option = options[ i ];
+
+					/* eslint-disable no-cond-assign */
+
 					if ( option.selected =
@@ -152,2 +155,4 @@ jQuery.inArray( jQuery.valHooks.option.get( option ), values ) > -1
 					}
+
+					/* eslint-enable no-cond-assign */
 				}
@@ -154,0 +159,0 @@

src/core.js

@@ -0,1 +1,5 @@
+/* global Symbol */
+// Defining this global in .eslintrc would create a danger of using the global
+// unguarded in another place, it seems safer to define global only for this module
+
 define( [
@@ -23,3 +27,3 @@ "./var/arr",
 var
-	version = "3.0.0",
+	version = "3.1.0",
 
@@ -256,3 +260,7 @@ // Define a local copy of jQuery
 	isEmptyObject: function( obj ) {
+
+		/* eslint-disable no-unused-vars */
+		// See https://github.com/eslint/eslint/issues/6125
 		var name;
+
 		for ( name in obj ) {
@@ -447,11 +455,5 @@ return false;
 
-// JSHint would error on this code due to the Symbol not being defined in ES5.
-// Defining this global in .jshintrc would create a danger of using the global
-// unguarded in another place, it seems safer to just disable JSHint for these
-// three lines.
-/* jshint ignore: start */
 if ( typeof Symbol === "function" ) {
 	jQuery.fn[ Symbol.iterator ] = arr[ Symbol.iterator ];
 }
-/* jshint ignore: end */
 
@@ -458,0 +460,0 @@ // Populate the class2type map

src/core/ready.js

 define( [
 	"../core",
 	"../var/document",
+	"../core/readyException",
 	"../deferred"
@@ -14,4 +15,12 @@ ], function( jQuery, document ) {
 
-	readyList.then( fn );
+	readyList
+		.then( fn )
 
+		// Wrap jQuery.readyException in a function so that the lookup
+		// happens at the time of error handling instead of callback
+		// registration.
+		.catch( function( error ) {
+			jQuery.readyException( error );
+		} );
+
 	return this;
@@ -18,0 +27,0 @@ };

src/css/adjustCSS.js

@@ -13,4 +13,8 @@ define( [
 		currentValue = tween ?
-			function() { return tween.cur(); } :
-			function() { return jQuery.css( elem, prop, "" ); },
+			function() {
+				return tween.cur();
+			} :
+			function() {
+				return jQuery.css( elem, prop, "" );
+			},
 		initial = currentValue(),
@@ -17,0 +21,0 @@ unit = valueParts && valueParts[ 3 ] || ( jQuery.cssNumber[ prop ] ? "" : "px" ),

src/data/var/acceptData.js

@@ -16,3 +16,2 @@ define( function() {
 	//    - Any
-	/* jshint -W018 */
 	return owner.nodeType === 1 || owner.nodeType === 9 || !( +owner.nodeType );
@@ -19,0 +18,0 @@ };

src/deferred.js

@@ -40,3 +40,3 @@ define( [
 	// Deferred#then to conditionally suppress rejection.
-	} catch ( /*jshint -W002 */ value ) {
+	} catch ( value ) {
 
@@ -43,0 +43,0 @@ // Support: Android 4.0 only

src/effects.js

@@ -54,3 +54,3 @@ define( [
 	includeWidth = includeWidth ? 1 : 0;
-	for ( ; i < 4 ; i += 2 - includeWidth ) {
+	for ( ; i < 4; i += 2 - includeWidth ) {
 		which = cssExpand[ i ];
@@ -82,3 +82,2 @@ attrs[ "margin" + which ] = attrs[ "padding" + which ] = type;
 function defaultPrefilter( elem, props, opts ) {
-	/* jshint validthis: true */
 	var prop, value, toggle, hooks, oldfire, propTween, restoreDisplay, display,
@@ -225,5 +224,8 @@ isBox = "width" in props || "height" in props,
 
-			/* jshint -W083 */
+			/* eslint-disable no-loop-func */
+
 			anim.done( function() {
 
+			/* eslint-enable no-loop-func */
+
 				// The final step of a "hide" animation is actually hiding the element
@@ -313,3 +315,3 @@ if ( !hidden ) {
 
-			for ( ; index < length ; index++ ) {
+			for ( ; index < length; index++ ) {
 				animation.tweens[ index ].run( percent );
@@ -355,3 +357,3 @@ }
 				stopped = true;
-				for ( ; index < length ; index++ ) {
+				for ( ; index < length; index++ ) {
 					animation.tweens[ index ].run( 1 );
@@ -374,3 +376,3 @@ }
 
-	for ( ; index < length ; index++ ) {
+	for ( ; index < length; index++ ) {
 		result = Animation.prefilters[ index ].call( animation, elem, props, animation.opts );
@@ -429,3 +431,3 @@ if ( result ) {
 
-		for ( ; index < length ; index++ ) {
+		for ( ; index < length; index++ ) {
 			prop = props[ index ];
@@ -432,0 +434,0 @@ Animation.tweeners[ prop ] = Animation.tweeners[ prop ] || [];

src/exports/global.js

@@ -5,3 +5,3 @@ /* ExcludeStart */
 // so the "use strict" pragma is not needed.
-/* jshint strict: false */
+/* eslint strict: "off" */
 
@@ -8,0 +8,0 @@ /* ExcludeEnd */

src/manipulation.js

@@ -32,4 +32,10 @@ define( [
 var
+
+	/* eslint-disable max-len */
+
+	// See https://github.com/eslint/eslint/issues/3229
 	rxhtmlTag = /<(?!area|br|col|embed|hr|img|input|link|meta|param)(([a-z][^\/\0>\x20\t\r\n\f]*)[^>]*)\/>/gi,
 
+	/* eslint-enable */
+
 	// Support: IE <=10 - 11, Edge 12 - 13
@@ -36,0 +42,0 @@ // In IE/Edge using regex groups here causes severe slowdowns.

src/selector.js

@@ -1,1 +0,3 @@
-define( [ "./selector-sizzle" ], function() {} );
+define( [ "./selector-sizzle" ], function() {
+	"use strict";
+} );

src/traversing/findFilter.js

@@ -16,3 +16,2 @@ define( [
 		return jQuery.grep( elements, function( elem, i ) {
-			/* jshint -W018 */
 			return !!qualifier.call( elem, i, elem ) !== not;
@@ -19,0 +18,0 @@ } );

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

1229013

increased by0.1%

Number of package files

117

increased by0.86%

Lines of code

24899

increased by0.26%

No dependency changes