Docs: Indicate which deprecated APIs are removed in 4.0; fix typos, etc.

Author: mgol

.editorconfig

@@ -10,6 +10,6 @@ charset = utf-8
 trim_trailing_whitespace = true
 insert_final_newline = true
 
-[*.{yml}]
+[*.yml]
 indent_style = space
 indent_size = 2

src/disablePatches.js

@@ -3,7 +3,7 @@
 var disabledPatches = Object.create( null );
 
 // Don't apply patches for specified codes. Helpful for code bases
-// where some Migrate warnings have been addressed and it's desirable
+// where some Migrate warnings have been addressed, and it's desirable
 // to avoid needless patches or false positives.
 jQuery.migrateDisablePatches = function() {
 	var i;

src/jquery/attributes.js

@@ -152,7 +152,7 @@ migratePatchFunc( jQuery.fn, "toggleClass", function( state ) {
 
 		// If the element has a class name or if we're passed `false`,
 		// then remove the whole classname (if there was one, the above saved it).
-		// Otherwise bring back whatever was previously saved (if anything),
+		// Otherwise, bring back whatever was previously saved (if anything),
 		// falling back to the empty string if nothing was stored.
 		if ( this.setAttribute ) {
 			this.setAttribute( "class",

src/main.js

@@ -28,13 +28,13 @@ window.console.log( "JQMIGRATE: Migrate is installed" +
 
 var warnedAbout = {};
 
-// By default each warning is only reported once.
+// By default, each warning is only reported once.
 jQuery.migrateDeduplicateWarnings = true;
 
 // List of warnings already given; public read only
 jQuery.migrateWarnings = [];
 
-// Set to false to disable traces that appear with warnings
+// Set to `false` to disable traces that appear with warnings
 if ( jQuery.migrateTrace === undefined ) {
 	jQuery.migrateTrace = true;
 }

src/utils.js

@@ -6,8 +6,8 @@ export function camelCase( string ) {
 	} );
 }
 
-// Insert an additional object in the prototype chain between `objrvy`
-// and `Object.prototype`; that intermediate object proxies properties
+// Make `object` inherit from `Object.prototype` via an additional object
+// in between; that intermediate object proxies properties
 // to `Object.prototype`, warning about their usage first.
 export function patchProto( object, options ) {
 

test/data/test-utils.js

@@ -17,15 +17,18 @@ window.expectWarning = function expectWarning( assert, name, expected, fn ) {
 		if ( expected === 0 ) {
 			assert.deepEqual( jQuery.migrateWarnings, [], name + ": did not warn" );
 
-		// Simple numeric equality assertion for warnings matching an explicit count
+		// Simple numeric equality assertion for warnings matching an explicit
+		// count
 		} else if ( expected && jQuery.migrateWarnings.length === expected ) {
 			assert.equal( jQuery.migrateWarnings.length, expected, name + ": warned" );
 
-		// Simple ok assertion when we saw at least one warning and weren't looking for an explict count
+		// Simple ok assertion when we saw at least one warning and weren't
+		// looking for an explicit count
 		} else if ( !expected && jQuery.migrateWarnings.length ) {
 			assert.ok( true, name + ": warned" );
 
-		// Failure; use deepEqual to show the warnings that *were* generated and the expectation
+		// Failure; use deepEqual to show the warnings that *were* generated
+		// and the expectation
 		} else {
 			assert.deepEqual( jQuery.migrateWarnings,
 				"<warnings: " + ( expected || "1+" ) + ">", name + ": warned"

test/unit/jquery/css.js

@@ -147,7 +147,7 @@ QUnit[
 	} );
 
 	// z-index is tested explicitly as raw jQuery 4.0 will not have `jQuery.cssNumber`
-	// so iterating over it won't find anything and we'd like to ensure number values
+	// so iterating over it won't find anything, and we'd like to ensure number values
 	// are not warned against for safe CSS props like z-index (gh-438).
 	expectNoWarning( assert, "z-index", function() {
 		jQuery( "<div />" ).css( "z-index", 1 );