Continuous Integration

Dynamic Sitemap Copy Plugin (#1232) #2175

Sign in to view logs

Summary
Jobs
- build (18)
Run details
- Usage
- Workflow file

Triggered via pull request July 27, 2024 13:01

jstockdi

synchronize #1260

jstockdi:1232-dynamic-sitemap

Status Failure

Total duration 1m 47s

Artifacts –

ci.yml

on: pull_request

Matrix: build

Annotations

10 errors and 11 warnings

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L1

'checkResourceExists' is defined but never used

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L1

Strings must use singlequote

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L4

More than 1 blank line not allowed

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L6

Expected indentation of 2 spaces but found 4

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L7

Expected indentation of 4 spaces but found 6

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L8

Expected indentation of 4 spaces but found 6

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L9

Trailing spaces not allowed

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L10

Expected indentation of 4 spaces but found 6

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L11

Expected indentation of 4 spaces but found 6

build (18): packages/cli/src/plugins/copy/plugin-copy-sitemap.js#L12

Expected indentation of 4 spaces but found 6

The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

build (18): packages/cli/src/commands/build.js#L8

Unexpected 'todo' comment: 'TODO a lot of these are duplicated in...'

build (18): packages/cli/src/commands/build.js#L8

Unexpected ' TODO' comment: 'TODO a lot of these are duplicated in...'

build (18): packages/cli/src/commands/build.js#L45

Unexpected 'todo' comment: 'TODO does this make more sense in bundle...'

build (18): packages/cli/src/commands/build.js#L45

Unexpected ' TODO' comment: 'TODO does this make more sense in bundle...'

build (18): packages/cli/src/config/rollup.config.js#L274

Unexpected 'todo' comment: 'TODO could we use this instead?'

build (18): packages/cli/src/config/rollup.config.js#L274

Unexpected ' TODO' comment: 'TODO could we use this instead?'

build (18): packages/cli/src/config/rollup.config.js#L379

Unexpected 'todo' comment: 'TODO should routes and APIs have chunks?'

build (18): packages/cli/src/config/rollup.config.js#L379

Unexpected ' TODO' comment: 'TODO should routes and APIs have chunks?'

build (18): packages/cli/src/config/rollup.config.js#L401

Unexpected 'todo' comment: 'TODO should routes and APIs have chunks?'

build (18): packages/cli/src/config/rollup.config.js#L401

Unexpected ' TODO' comment: 'TODO should routes and APIs have chunks?'