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.

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

Code Quality: Fix some misspelled words #45222

Merged
merged 1 commit into from
Oct 24, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/upload-release-to-plugin-repo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion bin/packages/watch.js
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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`

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion packages/block-editor/src/store/test/selectors.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
2 changes: 1 addition & 1 deletion packages/block-library/src/table/state.js
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion packages/components/src/external-link/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -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( '#' );
Expand Down
4 changes: 2 additions & 2 deletions packages/components/src/item-group/test/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -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
);
} );
} );
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 );
Expand Down
2 changes: 1 addition & 1 deletion packages/core-data/src/batch/default-processor.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 ) {
Expand Down
2 changes: 1 addition & 1 deletion packages/docgen/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
40 changes: 20 additions & 20 deletions packages/e2e-tests/specs/site-editor/multi-entity-saving.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down Expand Up @@ -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' );
Expand All @@ -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(
Expand All @@ -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.
Expand All @@ -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"]' );
Expand All @@ -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 () => {
Expand Down Expand Up @@ -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 );
Expand Down Expand Up @@ -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();
Expand Down
2 changes: 1 addition & 1 deletion packages/edit-navigation/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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.

Expand Down
2 changes: 1 addition & 1 deletion packages/env/lib/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 );
}
}
Expand Down
4 changes: 2 additions & 2 deletions packages/env/lib/config/validate-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 {}
Expand All @@ -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 ) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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,
},
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 ),
},
];
Expand Down
2 changes: 1 addition & 1 deletion packages/project-management-automation/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion packages/react-native-aztec/src/AztecView.js
Original file line number Diff line number Diff line change
Expand Up @@ -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.
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion packages/url/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down