Skip to content
Linting

WIP Implement media import via URL #67697

Sign in to view logs

WIP Implement media import via URL

WIP Implement media import via URL #67697

Triggered via pull request January 16, 2025 07:56
@fusharfushar
synchronize #41089
wpcom-media-url-import
Status Failure
Total duration 3m 34s
Artifacts –

linting.yml

on: pull_request
detect changed files
7s
detect changed files
Copied files are in sync
7s
Copied files are in sync
Changelogger use
19s
Changelogger use
Changelogger validity
24s
Changelogger validity
Project structure
1m 18s
Project structure
PHP Code Sniffer (non-excluded files only)
1m 48s
PHP Code Sniffer (non-excluded files only)
PHP Compatibility
1m 12s
PHP Compatibility
PHP Code Sniffer (changes to excluded files only)
0s
PHP Code Sniffer (changes to excluded files only)
ESLint (non-excluded files only)
3m 10s
ESLint (non-excluded files only)
ESLint (changes to excluded files only)
0s
ESLint (changes to excluded files only)
Lint GitHub Actions yaml files
0s
Lint GitHub Actions yaml files
Check linter exclude lists
0s
Check linter exclude lists
Lock files are up to date
0s
Lock files are up to date
Monorepo package version refs
0s
Monorepo package version refs
Matrix: php_lint
Fit to window
Zoom out
Zoom in

Annotations

19 errors and 20 warnings
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L6
There must be exactly one blank line before the tags in a doc comment (Generic.Commenting.DocComment.SpacingBeforeTags)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L12
Missing doc comment for function wpcom_media_url_import() (Squiz.Commenting.FunctionComment.Missing)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L13
Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L14
Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L15
Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L16
Tabs must be used to indent lines; spaces are not allowed (Generic.WhiteSpace.DisallowSpaceIndent.SpacesUsed)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L18
Expected 1 spaces after opening parenthesis; 0 found (PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L38
Expected 1 spaces after opening parenthesis; 0 found (PEAR.Functions.FunctionCallSignature.SpaceAfterOpenBracket)
PHP Code Sniffer (non-excluded files only)
Process completed with exit code 2.
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L8
Replace `····` with `↹↹`
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L9
Replace `··` with `↹`
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L13
Do not use 'new' for side effects
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L13
Replace `url` with `·url·`
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L14
Replace `e` with `·e·`
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L14
'e' is defined but never used
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L14
'e' is already declared in the upper scope on line 11 column 29
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L19
Unexpected console statement
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L19
Insert `·`
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L21
Unexpected console statement
Copied files are in sync
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
detect changed files
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Changelogger use
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Changelogger validity
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (7.2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (8.3)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (8.4)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (7.4)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (8.2)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (8.0)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP lint (8.1)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Project structure
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP Compatibility
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP Code Sniffer (non-excluded files only)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L9
Use statements are not alphabetically sorted (MediaWiki.Classes.UnsortedUseStatements.UnsortedUse)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L9
Unused use statement "Jetpack_Connection" (MediaWiki.Classes.UnusedUseStatement.UnusedUse)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L10
Unused use statement "Client" (MediaWiki.Classes.UnusedUseStatement.UnusedUse)
PHP Code Sniffer (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import.php#L27
Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space (Generic.Formatting.MultipleStatementAlignment.NotSameWarning)
ESLint (non-excluded files only)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
ESLint (non-excluded files only): projects/packages/jetpack-mu-wpcom/src/features/wpcom-media/wpcom-media-url-import-form/index.jsx#L4
Missing JSDoc comment