Skip to content
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
4 changes: 0 additions & 4 deletions .github/scripts/android-toolchain-sync.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,6 @@
* reconciles a single tracking issue. Run by
* `.github/workflows/android-toolchain-sync.yml`.
*/

/**
* External dependencies
*/
import { readFile } from 'fs/promises';

const UPSTREAM = 'wordpress-mobile/WordPress-Android';
Expand Down
8 changes: 0 additions & 8 deletions bin/check-wordpress-package-duplicates.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,17 +17,9 @@
* install paths distinguish separate copies of the same version, which are
* still separate module instances.
*/

/**
* External dependencies
*/
import { readFileSync } from 'fs';
import { dirname, join } from 'path';
import { fileURLToPath } from 'url';

/**
* Internal dependencies
*/
import { error, info, warn } from '../src/utils/logger.js';

const rootDir = join( dirname( fileURLToPath( import.meta.url ) ), '..' );
Expand Down
7 changes: 0 additions & 7 deletions bin/fetch-translations.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import fs from 'fs';
import path from 'path';

/**
* Internal dependencies
*/
import { info, error, debug } from '../src/utils/logger.js';

// Uses Node's built-in `fetch` (Node 20, per .nvmrc) so this script can
Expand Down
8 changes: 0 additions & 8 deletions bin/generate-version.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,9 @@
* Generate version files for iOS and Android from package.json
* This ensures a single source of truth for the version number
*/

/**
* External dependencies
*/
import { readFileSync, writeFileSync } from 'fs';
import { fileURLToPath } from 'url';
import { dirname, join } from 'path';

/**
* Internal dependencies
*/
import { info } from '../src/utils/logger.js';

const __filename = fileURLToPath( import.meta.url );
Expand Down
7 changes: 0 additions & 7 deletions e2e/audio-upload.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';
import { uploadsPathPattern } from './wp-env-fixtures';

Expand Down
7 changes: 0 additions & 7 deletions e2e/block-reorder.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Block Reorder', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/buttons-block.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Buttons Block', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/color-gradient.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Color and Gradient', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/cover-block.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Cover Block', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/editor-content-changed.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'onEditorContentChanged', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/editor-error.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Editor Error Handling', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/editor-load.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Editor Load', () => {
Expand Down
4 changes: 0 additions & 4 deletions e2e/editor-page.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@
* Encapsulates common editor interactions — setup, block appending,
* caret movement, and data-store queries — behind a single class.
*/

/**
* Internal dependencies
*/
import {
credentials,
getEditorSettings,
Expand Down
7 changes: 0 additions & 7 deletions e2e/editor-title.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Editor Title', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/embed-content.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';
import { credentials } from './wp-env-fixtures';

Expand Down
7 changes: 0 additions & 7 deletions e2e/fetch-json.test.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';

/**
* Internal dependencies
*/
import { fetchJson } from './fetch-json';

const CREDS = { authHeader: 'Basic dGVzdDp0ZXN0' };
Expand Down
7 changes: 0 additions & 7 deletions e2e/file-upload.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';
import { uploadsPathPattern } from './wp-env-fixtures';

Expand Down
7 changes: 0 additions & 7 deletions e2e/gallery-block.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

const TEST_IMAGE_1 = path.resolve(
Expand Down
7 changes: 0 additions & 7 deletions e2e/get-title-and-content.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'getTitleAndContent', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/image-upload.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';
import { uploadsPathPattern } from './wp-env-fixtures';

Expand Down
7 changes: 0 additions & 7 deletions e2e/link-formatting.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Link Formatting', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/shortcode-block.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Shortcode Block', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/split-merge.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

/**
Expand Down
7 changes: 0 additions & 7 deletions e2e/text-alignment.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Text Alignment', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/text-formatting.spec.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,4 @@
/**
* External dependencies
*/
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

test.describe( 'Text Formatting', () => {
Expand Down
7 changes: 0 additions & 7 deletions e2e/third-party-blocks.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';

const TEST_IMAGE_1 = path.resolve(
Expand Down
7 changes: 0 additions & 7 deletions e2e/video-upload.spec.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import path from 'node:path';
import { test, expect } from '@playwright/test';

/**
* Internal dependencies
*/
import EditorPage from './editor-page';
import { uploadsPathPattern } from './wp-env-fixtures';

Expand Down
7 changes: 0 additions & 7 deletions e2e/wp-env-fixtures.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
/**
* External dependencies
*/
import fs from 'node:fs';
import path from 'node:path';

/**
* Internal dependencies
*/
import { fetchJson } from './fetch-json';

const CREDENTIALS_PATH = path.resolve(
Expand Down
19 changes: 16 additions & 3 deletions eslint.config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
/**
* External dependencies
*/
import { defineConfig, globalIgnores } from 'eslint/config';
import wordpress from '@wordpress/eslint-plugin';
import reactRefresh from 'eslint-plugin-react-refresh';
Expand Down Expand Up @@ -43,6 +40,22 @@ export default defineConfig( [
'react-refresh': reactRefresh,
},
rules: {
// Imports form one contiguous block, aligned with the upstream
// Gutenberg ESLint configuration.
'@wordpress/dependency-group': [ 'error', 'never' ],
'import/order': [
'error',
{
groups: [
'builtin',
'external',
'internal',
[ 'parent', 'sibling', 'index' ],
],
'newlines-between': 'never',
warnOnUnassignedImports: true,
},
],
'react-refresh/only-export-components': [
'warn',
{ allowConstantExport: true },
Expand Down
3 changes: 0 additions & 3 deletions playwright.config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
/**
* External dependencies
*/
import { defineConfig, devices } from '@playwright/test';

const isCI = process.env.CI === 'true';
Expand Down
Loading
Loading