Skip to content

feat: STRF-11903 Migrate to ESM #917

feat: STRF-11903 Migrate to ESM

feat: STRF-11903 Migrate to ESM #917

Triggered via pull request September 12, 2024 09:56
Status Failure
Total duration 51s
Artifacts

pull_request_review.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

33 errors and 20 warnings
build (18.x, ubuntu-latest): lib/BuildConfigManager.js#L47
Calls to require() should use string literals
build (18.x, ubuntu-latest): lib/BuildConfigManager.js#L48
Unnecessary 'else' after 'return'
build (18.x, ubuntu-latest): lib/BuildConfigManager.js#L50
Calls to require() should use string literals
build (18.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (18.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
"@jest/globals" is extraneous
build (18.x, ubuntu-latest): lib/StencilConfigManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (18.x, ubuntu-latest): lib/StencilConfigManager.spec.js#L1
"@jest/globals" is extraneous
build (18.x, ubuntu-latest): lib/StencilDebug.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (18.x, ubuntu-latest): lib/StencilDebug.spec.js#L1
"@jest/globals" is extraneous
build (18.x, ubuntu-latest): lib/archiveManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (20.x, ubuntu-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (20.x, ubuntu-latest): lib/BuildConfigManager.js#L47
Calls to require() should use string literals
build (20.x, ubuntu-latest): lib/BuildConfigManager.js#L48
Unnecessary 'else' after 'return'
build (20.x, ubuntu-latest): lib/BuildConfigManager.js#L50
Calls to require() should use string literals
build (20.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (20.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
"@jest/globals" is extraneous
build (20.x, ubuntu-latest): lib/StencilConfigManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (20.x, ubuntu-latest): lib/StencilConfigManager.spec.js#L1
"@jest/globals" is extraneous
build (20.x, ubuntu-latest): lib/StencilDebug.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (20.x, ubuntu-latest): lib/StencilDebug.spec.js#L1
"@jest/globals" is extraneous
build (20.x, ubuntu-latest): lib/archiveManager.spec.js#L1
'@jest/globals' should be listed in the project's dependencies. Run 'npm i -S @jest/globals' to add it
build (18.x, windows-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (18.x, windows-latest)
The operation was canceled.
build (20.x, windows-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (20.x, windows-latest)
The operation was canceled.
build (22.x, ubuntu-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (22.x, ubuntu-latest)
The operation was canceled.
build (20.x, macos-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (20.x, macos-latest)
The operation was canceled.
build (18.x, macos-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (18.x, macos-latest)
The operation was canceled.
build (22.x, macos-latest)
The job was canceled because "_18_x_ubuntu-latest" failed.
build (22.x, macos-latest)
The operation was canceled.
build (18.x, ubuntu-latest): lib/BuildConfigManager.js#L5
Replace `"node:module"` with `'node:module'`
build (18.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
Replace `jest` with `·jest·`
build (18.x, ubuntu-latest): lib/bundle-validator.js#L29
Replace `⏎····fs.readFileSync(privateThemeConfigUrl),⏎` with `fs.readFileSync(privateThemeConfigUrl)`
build (18.x, ubuntu-latest): lib/lang-assembler.spec.js#L1
Replace `jest` with `·jest·`
build (18.x, ubuntu-latest): lib/stencil-start.spec.js#L1
Replace `jest` with `·jest·`
build (18.x, ubuntu-latest): lib/theme-config.spec.js#L13
Replace `fs.readFileSync(path.join(__dirname,·'../test/_mocks/themes/valid/schema.json'))` with `⏎····fs.readFileSync(path.join(__dirname,·'../test/_mocks/themes/valid/schema.json')),⏎`
build (18.x, ubuntu-latest): lib/validator/schema-translations.spec.js#L8
Replace `readFileSync(`${process.cwd()}/lib/schemas/schemaTranslations.json`)` with `⏎····readFileSync(`${process.cwd()}/lib/schemas/schemaTranslations.json`),⏎`
build (18.x, ubuntu-latest): server/index.js#L9
Delete `····`
build (18.x, ubuntu-latest): server/index.js#L24
Insert `⏎·······`
build (20.x, ubuntu-latest): lib/BuildConfigManager.js#L5
Replace `"node:module"` with `'node:module'`
build (20.x, ubuntu-latest): lib/BuildConfigManager.spec.js#L1
Replace `jest` with `·jest·`
build (20.x, ubuntu-latest): lib/bundle-validator.js#L29
Replace `⏎····fs.readFileSync(privateThemeConfigUrl),⏎` with `fs.readFileSync(privateThemeConfigUrl)`
build (20.x, ubuntu-latest): lib/lang-assembler.spec.js#L1
Replace `jest` with `·jest·`
build (20.x, ubuntu-latest): lib/stencil-start.spec.js#L1
Replace `jest` with `·jest·`
build (20.x, ubuntu-latest): lib/theme-config.spec.js#L13
Replace `fs.readFileSync(path.join(__dirname,·'../test/_mocks/themes/valid/schema.json'))` with `⏎····fs.readFileSync(path.join(__dirname,·'../test/_mocks/themes/valid/schema.json')),⏎`
build (20.x, ubuntu-latest): lib/validator/schema-translations.spec.js#L8
Replace `readFileSync(`${process.cwd()}/lib/schemas/schemaTranslations.json`)` with `⏎····readFileSync(`${process.cwd()}/lib/schemas/schemaTranslations.json`),⏎`
build (20.x, ubuntu-latest): server/index.js#L9
Delete `····`
build (20.x, ubuntu-latest): server/index.js#L24
Insert `⏎·······`