Code Quality: Fix some misspelled words (#45222)

.github/workflows/upload-release-to-plugin-repo.yml

@@ -79,7 +79,7 @@ jobs:
             - name: Commit the Changelog update
               run: |
                   git add changelog.txt
-                  # Remove files that are not meant to be commited
+                  # Remove files that are not meant to be committed
                   # ie. release_notes.txt created on the previous step.
                   git clean -fd
                   # Only attempt to commit changelog if it has been modified.

bin/packages/watch.js

@@ -176,7 +176,7 @@ watch(
 	PACKAGES_DIR,
 	{ recursive: true, delay: 500, filter: isWatchableFile },
 	( event, filename ) => {
-		// Double check whether we're dealing with a file that needs watching, to accomodate for
+		// Double check whether we're dealing with a file that needs watching, to accommodate for
 		// the inability to watch recursively on linux-based operating systems.
 		if ( ! isSourceFile( filename ) || ! isModulePackage( filename ) ) {
 			return;

packages/block-editor/src/components/block-list/style.scss

@@ -174,7 +174,7 @@
 	// With `position: static`, Safari marks a full-width selection rectangle, including margins.
 	// With `position: relative`, Safari marks an inline selection rectangle, similar to that of
 	// Blink based browsers, but it also does "crop" the marker, which can result in a small line
-	// from the preceeding paragraph showing, which is effectively the above selection bleeding in.
+	// from the preceding paragraph showing, which is effectively the above selection bleeding in.
 	// We choose relative, as that matches the multi-selection, which is limited to the block footprint.
 	position: relative;
 

packages/block-editor/src/components/block-mover/stories/index.js

@@ -70,7 +70,7 @@ function BlockMoverStory() {
 			</Toolbar>
 
 			<p style={ { marginTop: 36 } }>
-				But it can also accomodate horizontal blocks.
+				But it can also accommodate horizontal blocks.
 			</p>
 			<Toolbar label="Block Mover">
 				<BlockMover

packages/block-editor/src/components/block-variation-picker/README.md

@@ -68,7 +68,7 @@ The different variations of the block.
 
 -   Type: `Function`
 
-Callback called when a block variation is selected. It recieves the selected variation as a parameter.
+Callback called when a block variation is selected. It receives the selected variation as a parameter.
 
 #### `icon`
 

packages/block-editor/src/components/rich-text/use-paste-handler.js

@@ -255,7 +255,7 @@ export function usePasteHandler( props ) {
 
 /**
  * Normalizes a given string of HTML to remove the Windows-specific "Fragment"
- * comments and any preceeding and trailing content.
+ * comments and any preceding and trailing content.
  *
  * @param {string} html the html to be normalized
  * @return {string} the normalized html

packages/block-editor/src/store/test/selectors.js

@@ -3995,7 +3995,7 @@ describe( 'selectors', () => {
 } );
 
 describe( 'getInserterItems with core blocks prioritization', () => {
-	// This test is in a seperate `describe` because all other tests register
+	// This test is in a separate `describe` because all other tests register
 	// some test `core` blocks and interfere with the purpose of the specific test.
 	// This tests the functionality to ensure core blocks are prioritized in the
 	// returned results, because third party blocks can be registered earlier than

packages/block-library/src/table/state.js

@@ -281,7 +281,7 @@ export function deleteColumn( state, { columnIndex } ) {
 }
 
 /**
- * Toggles the existance of a section.
+ * Toggles the existence of a section.
  *
  * @param {Object} state       Current table state.
  * @param {string} sectionName Name of the section to toggle.

packages/components/src/external-link/index.tsx

@@ -38,7 +38,7 @@ function UnforwardedExternalLink(
 		),
 	].join( ' ' );
 	const classes = classnames( 'components-external-link', className );
-	/* Anchor links are percieved as external links.
+	/* Anchor links are perceived as external links.
 	This constant helps check for on page anchor links,
 	to prevent them from being opened in the editor. */
 	const isInternalAnchor = !! href?.startsWith( '#' );

packages/components/src/item-group/test/index.js

@@ -65,14 +65,14 @@ describe( 'ItemGroup', () => {
 				</ItemGroup>
 			);
 
-			const { container: seperatedItems } = render(
+			const { container: separatedItems } = render(
 				<ItemGroup isSeparated={ true }>
 					<Item>Code is poetry</Item>
 				</ItemGroup>
 			);
 
 			expect( groupedItems.firstChild ).toMatchDiffSnapshot(
-				seperatedItems.firstChild
+				separatedItems.firstChild
 			);
 		} );
 	} );

packages/components/src/slot-fill/bubbles-virtually/fill.js

@@ -32,7 +32,7 @@ export default function Fill( { name, children } ) {
 	const ref = useRef( { rerender: useForceUpdate() } );
 
 	useEffect( () => {
-		// We register fills so we can keep track of their existance.
+		// We register fills so we can keep track of their existence.
 		// Some Slot implementations need to know if there're already fills
 		// registered so they can choose to render themselves or not.
 		registerFill( ref );

packages/core-data/src/batch/default-processor.js

@@ -27,7 +27,7 @@ function chunk( arr, chunkSize ) {
  * @param {Array} requests List of API requests to perform at once.
  *
  * @return {Promise} Promise that resolves to a list of objects containing
- *                   either `output` (if that request was succesful) or `error`
+ *                   either `output` (if that request was successful) or `error`
  *                   (if not ).
  */
 export default async function defaultProcessor( requests ) {

packages/docgen/CHANGELOG.md

@@ -47,7 +47,7 @@
 
 ### Internal
 
--   Remove unneccessary argument from an instance of `Array#pop`.
+-   Remove unnecessary argument from an instance of `Array#pop`.
 
 ## 1.0.0 (2019-03-06)
 

packages/e2e-tests/specs/editor/blocks/navigation.test.js

@@ -506,7 +506,7 @@ Expected mock function not to be called but it was called with: ["POST", "http:/
 				// outside of the Nav block.
 				await insertBlock( 'Paragraph' );
 
-				// Aquire fresh reference to block
+				// Acquire fresh reference to block
 				navBlock = await waitForBlock( 'Navigation' );
 
 				// Check Placeholder Preview is visible.

packages/e2e-tests/specs/site-editor/multi-entity-saving.test.js

@@ -31,7 +31,7 @@ describe( 'Multi-entity save flow', () => {
 		const checkboxInputs = await page.$$( checkboxInputSelector );
 		expect( checkedBoxes.length - checkboxInputs.length ).toBe( 0 );
 	};
-	const assertExistance = async ( selector, shouldBePresent ) => {
+	const assertExistence = async ( selector, shouldBePresent ) => {
 		const element = await page.$( selector );
 		if ( shouldBePresent ) {
 			expect( element ).not.toBeNull();
@@ -102,10 +102,10 @@ describe( 'Multi-entity save flow', () => {
 			await assertMultiSaveDisabled();
 
 			// Should only have publish panel a11y button active with only post edited.
-			await assertExistance( publishA11ySelector, true );
-			await assertExistance( saveA11ySelector, false );
-			await assertExistance( publishPanelSelector, false );
-			await assertExistance( savePanelSelector, false );
+			await assertExistence( publishA11ySelector, true );
+			await assertExistence( saveA11ySelector, false );
+			await assertExistence( publishPanelSelector, false );
+			await assertExistence( savePanelSelector, false );
 
 			// Add a reusable block and edit it.
 			await createReusableBlock( 'Hi!', 'Test' );
@@ -117,29 +117,29 @@ describe( 'Multi-entity save flow', () => {
 			await assertMultiSaveEnabled();
 
 			// Should only have save panel a11y button active after child entities edited.
-			await assertExistance( publishA11ySelector, false );
-			await assertExistance( saveA11ySelector, true );
-			await assertExistance( publishPanelSelector, false );
-			await assertExistance( savePanelSelector, false );
+			await assertExistence( publishA11ySelector, false );
+			await assertExistence( saveA11ySelector, true );
+			await assertExistence( publishPanelSelector, false );
+			await assertExistence( savePanelSelector, false );
 
 			// Opening panel has boxes checked by default.
 			await page.click( savePostSelector );
 			await page.waitForSelector( savePanelSelector );
 			await assertAllBoxesChecked();
 
 			// Should not show other panels (or their a11y buttons) while save panel opened.
-			await assertExistance( publishA11ySelector, false );
-			await assertExistance( saveA11ySelector, false );
-			await assertExistance( publishPanelSelector, false );
+			await assertExistence( publishA11ySelector, false );
+			await assertExistence( saveA11ySelector, false );
+			await assertExistence( publishPanelSelector, false );
 
 			// Publish panel should open after saving.
 			await page.click( entitiesSaveSelector );
 			await page.waitForSelector( publishPanelSelector );
 
 			// No other panels (or their a11y buttons) should be present with publish panel open.
-			await assertExistance( publishA11ySelector, false );
-			await assertExistance( saveA11ySelector, false );
-			await assertExistance( savePanelSelector, false );
+			await assertExistence( publishA11ySelector, false );
+			await assertExistence( saveA11ySelector, false );
+			await assertExistence( savePanelSelector, false );
 
 			// Close publish panel.
 			const closePanelButton = await page.waitForSelector(
@@ -151,7 +151,7 @@ describe( 'Multi-entity save flow', () => {
 			const draftSaved = await page.waitForSelector( draftSavedSelector );
 			expect( draftSaved ).not.toBeNull();
 			await assertMultiSaveDisabled();
-			await assertExistance( saveA11ySelector, false );
+			await assertExistence( saveA11ySelector, false );
 
 			await publishPost();
 			// Wait for the success notice specifically for the published post.
@@ -178,7 +178,7 @@ describe( 'Multi-entity save flow', () => {
 			expect( enabledSaveButton ).not.toBeNull();
 			// Verify multi-entity saving not enabled.
 			await assertMultiSaveDisabled();
-			await assertExistance( saveA11ySelector, false );
+			await assertExistence( saveA11ySelector, false );
 
 			// Update reusable block again.
 			await page.click( 'p[data-type="core/paragraph"]' );
@@ -188,7 +188,7 @@ describe( 'Multi-entity save flow', () => {
 
 			// Multi-entity saving should be enabled.
 			await assertMultiSaveEnabled();
-			await assertExistance( saveA11ySelector, true );
+			await assertExistence( saveA11ySelector, true );
 		} );
 
 		it( 'Site blocks should save individually', async () => {
@@ -252,7 +252,7 @@ describe( 'Multi-entity save flow', () => {
 			await assertAllBoxesChecked();
 
 			// Save a11y button should not be present with save panel open.
-			await assertExistance( saveA11ySelector, false );
+			await assertExistence( saveA11ySelector, false );
 
 			// Saving should result in items being saved.
 			await page.click( entitiesSaveSelector );
@@ -284,7 +284,7 @@ describe( 'Multi-entity save flow', () => {
 			expect( enabledButton ).not.toBeNull();
 
 			// Save a11y button should be present.
-			await assertExistance( saveA11ySelector, true );
+			await assertExistence( saveA11ySelector, true );
 
 			// Save all changes.
 			await saveAllChanges();

packages/edit-navigation/README.md

@@ -22,7 +22,7 @@ initialize( '#navigation-editor-root', blockEditorSettings );
 
 ## Purpose
 
-By default, the Navigation Editor screen allows users to create and edit complex navigations using a block-based UI. The aim is to supercede [the current Menus screen](https://codex.wordpress.org/WordPress_Menu_User_Guide) by providing a superior experience whilst retaining backwards compatibility.
+By default, the Navigation Editor screen allows users to create and edit complex navigations using a block-based UI. The aim is to supersede [the current Menus screen](https://codex.wordpress.org/WordPress_Menu_User_Guide) by providing a superior experience whilst retaining backwards compatibility.
 
 The editing experience is provided as a block editor wrapper around the core functionality of the **Navigation _block_**. Features of the block are disabled/enhanced as necessary to provide an experience appropriate to editing a navigation outside of a Full Site Editing context.
 

packages/env/lib/cli.js

@@ -71,7 +71,7 @@ const withSpinner =
 					console.error( error );
 					process.exit( 1 );
 				} else {
-					spinner.fail( 'An unknown error occured.' );
+					spinner.fail( 'An unknown error occurred.' );
 					process.exit( 1 );
 				}
 			}

packages/env/lib/config/validate-config.js

@@ -6,7 +6,7 @@
  */
 
 /**
- * Error subtype which indicates that an expected validation erorr occured
+ * Error subtype which indicates that an expected validation erorr occurred
  * while reading wp-env configuration.
  */
 class ValidationError extends Error {}
@@ -16,7 +16,7 @@ class ValidationError extends Error {}
  * do not match the required format.
  *
  * @param {Object}  config      A config object to validate.
- * @param {?string} envLocation Identifies if the error occured in a specific environment property.
+ * @param {?string} envLocation Identifies if the error occurred in a specific environment property.
  * @return {Object} The passed config object with no modifications.
  */
 function validateConfig( config, envLocation ) {

packages/eslint-plugin/rules/__tests__/data-no-store-string-literals.js

@@ -44,7 +44,7 @@ const createSuggestionTestCase = ( code, output ) => ( {
 		{
 			suggestions: [
 				{
-					desc: 'Replace literal with store definition. Import store if neccessary.',
+					desc: 'Replace literal with store definition. Import store if necessary.',
 					output,
 				},
 			],

packages/eslint-plugin/rules/data-no-store-string-literals.js

@@ -122,7 +122,7 @@ function collectAllNodesFromObjectPropertyFunctionCalls( context, node ) {
 function getSuggest( context, callNode ) {
 	return [
 		{
-			desc: 'Replace literal with store definition. Import store if neccessary.',
+			desc: 'Replace literal with store definition. Import store if necessary.',
 			fix: ( fixer ) => getFixes( fixer, context, callNode ),
 		},
 	];

packages/project-management-automation/CHANGELOG.md

@@ -13,7 +13,7 @@
 ### New feature
 
 -   Include TypeScript type declarations ([#18942](https://github.com/WordPress/gutenberg/pull/18942))
--   The "Add First Time Contributor Label" task now prompts the user to link their GitHub account to their WordPress.org profile if neccessary for props credit. The task has been renamed "First Time Contributor".
+-   The "Add First Time Contributor Label" task now prompts the user to link their GitHub account to their WordPress.org profile if necessary for props credit. The task has been renamed "First Time Contributor".
 
 ### Improvements
 

packages/project-management-automation/lib/tasks/first-time-contributor-account-link/index.js

@@ -34,7 +34,7 @@ function getPromptMessageText( author ) {
 
 /**
  * Prompts the user to link their GitHub account to their WordPress.org profile
- * if neccessary for props credit.
+ * if necessary for props credit.
  *
  * @param {WebhookPayloadPush} payload Push event payload.
  * @param {GitHub}             octokit Initialized Octokit REST client.

packages/react-native-aztec/src/AztecView.js

@@ -252,7 +252,7 @@ class AztecView extends Component {
 			window.console.warn(
 				"Removing lineHeight style as it's not supported by native AztecView"
 			);
-			// Prevents passing line-heigth within styles to avoid a crash due to values without units
+			// Prevents passing line-height within styles to avoid a crash due to values without units
 			// We now support this but passing line-height as a prop instead.
 		}
 

packages/react-native-bridge/ios/GutenbergWebFallback/GutenbergWebSingleBlockViewController.swift

@@ -163,7 +163,7 @@ extension GutenbergWebSingleBlockViewController: WKNavigationDelegate {
 
     public func webView(_ webView: WKWebView, didFinish navigation: WKNavigation!) {
         // Sometimes the editor takes longer loading and its CSS can override what
-        // Injectic Editor specific CSS when everything is loaded to avoid overwritting parameters if gutenberg CSS load later.
+        // Injectic Editor specific CSS when everything is loaded to avoid overwriting parameters if gutenberg CSS load later.
         evaluateJavascript(jsInjection.preventAutosavesScript)
         evaluateJavascript(jsInjection.injectEditorCssScript)
         evaluateJavascript(jsInjection.gutenbergObserverScript)

packages/react-native-editor/__device-tests__/gutenberg-editor-lists.test.js

@@ -16,7 +16,7 @@ describe( 'Gutenberg Editor tests for List block', () => {
 		// Send an Enter.
 		await editorPage.typeTextToTextBlock( listBlockElement, '\n', false );
 
-		// Instead of introducing separate conditions for local and CI environment, add this wait for Android to accomodate both environments
+		// Instead of introducing separate conditions for local and CI environment, add this wait for Android to accommodate both environments
 		await waitIfAndroid();
 
 		// Send a backspace.

packages/url/CHANGELOG.md

@@ -113,7 +113,7 @@
 ### Bug fixes
 
 -   The `isValidProtocol` function now correctly considers the protocol of the URL as only incoporating characters up to and including the colon (':').
--   `getFragment` is now greedier and matches fragments from the first occurence of the '#' symbol instead of the last.
+-   `getFragment` is now greedier and matches fragments from the first occurrence of the '#' symbol instead of the last.
 
 ## 2.3.0 (2018-11-12)