Skip to content

Commit

Permalink
Remove period from eslint rule disables
Browse files Browse the repository at this point in the history
  • Loading branch information
opr committed Feb 25, 2022
1 parent 6566efe commit a16e0ac
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1633,7 +1633,7 @@ describe( 'Selecting links', () => {
);

// We should have highlighted the first item using the keyboard
// eslint-disable-next-line jest/no-conditional-expect.
// eslint-disable-next-line jest/no-conditional-expect
expect( selectedSearchResultElement ).toEqual(
secondSearchSuggestion
);
Expand All @@ -1648,7 +1648,7 @@ describe( 'Selecting links', () => {
);

// We should be back to highlighting the first search result again
// eslint-disable-next-line jest/no-conditional-expect.
// eslint-disable-next-line jest/no-conditional-expect
expect( selectedSearchResultElement ).toEqual(
firstSearchSuggestion
);
Expand Down
2 changes: 1 addition & 1 deletion packages/components/src/modal/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ function Modal( props, forwardedRef ) {
}

return createPortal(
// eslint-disable-next-line jsx-a11y/no-static-element-interactions.
// eslint-disable-next-line jsx-a11y/no-static-element-interactions
<div
ref={ useMergeRefs( [ ref, forwardedRef ] ) }
className={ classnames(
Expand Down
2 changes: 1 addition & 1 deletion packages/components/src/utils/hooks/stories/use-cx.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ const Example = ( { serializedStyles, children } ) => {
const ExampleWithUseMemoWrong = ( { serializedStyles, children } ) => {
const cx = useCx();
// Wrong: using 'useMemo' without adding 'cx' to the dependency list.
// eslint-disable-next-line react-hooks/exhaustive-deps.
// eslint-disable-next-line react-hooks/exhaustive-deps
const classes = useMemo( () => cx( serializedStyles ), [
serializedStyles,
] );
Expand Down
2 changes: 1 addition & 1 deletion packages/e2e-tests/specs/editor/blocks/navigation.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ async function waitForBlock( blockName ) {
}

// Disable reason - these tests are to be re-written.
// eslint-disable-next-line jest/no-disabled-tests.
// eslint-disable-next-line jest/no-disabled-tests
describe( 'Navigation', () => {
const contributorUsername = uniqueId( 'contributoruser_' );
let contributorPassword;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ describe( 'Reusable blocks', () => {
await page.keyboard.type( ' modified' );

// Wait for async mode to dispatch the update.
// eslint-disable-next-line no-restricted-syntax.
// eslint-disable-next-line no-restricted-syntax
await page.waitForTimeout( 1000 );

// Check that the content of the second reusable block has been updated.
Expand Down
4 changes: 2 additions & 2 deletions test/integration/full-content/full-content.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import {
getBlockTypes,
parse,
serialize,
unstable__bootstrapServerSideBlockDefinitions, // eslint-disable-line camelcase.
unstable__bootstrapServerSideBlockDefinitions, // eslint-disable-line camelcase
} from '@wordpress/blocks';
import { parse as grammarParse } from '@wordpress/block-serialization-default-parser';
import {
Expand Down Expand Up @@ -82,7 +82,7 @@ describe( 'full post content fixture', () => {
}

blockBasenames.forEach( ( basename ) => {
// eslint-disable-next-line jest/valid-title.
// eslint-disable-next-line jest/valid-title
it( basename, () => {
const {
filename: htmlFixtureFileName,
Expand Down
2 changes: 1 addition & 1 deletion test/native/helpers.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import {
// Editor component is not exposed in the pacakge because is meant to be consumed
// internally, however we require it for rendering the editor in integration tests,
// for this reason it's imported with path access.
// eslint-disable-next-line no-restricted-syntax.
// eslint-disable-next-line no-restricted-syntax
import Editor from '@wordpress/edit-post/src/editor';

// Set up the mocks for getting the HTML output of the editor.
Expand Down

0 comments on commit a16e0ac

Please sign in to comment.