Skip to content

feat: STRF-10101 Check translation rows compilation on stencil bundle/start #809

feat: STRF-10101 Check translation rows compilation on stencil bundle/start

feat: STRF-10101 Check translation rows compilation on stencil bundle/start #809

Triggered via pull request November 7, 2023 14:06
Status Failure
Total duration 53s
Artifacts

pull_request_review.yml

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

Annotations

29 errors and 20 warnings
build (16.x, ubuntu-latest): lib/lang/validator.js#L34
'path' is already declared in the upper scope on line 3 column 7
build (16.x, ubuntu-latest): lib/lang/validator.js#L39
Do not access Object.prototype method 'hasOwnProperty' from target object
build (16.x, ubuntu-latest): lib/lang/validator.js#L57
Duplicate name 'validate'
build (16.x, ubuntu-latest): lib/lang/validator.js#L59
'path' is already declared in the upper scope on line 3 column 7
build (16.x, ubuntu-latest): lib/lang/validator.js#L60
for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array
build (16.x, ubuntu-latest): lib/lang/validator.js#L60
The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype
build (16.x, ubuntu-latest): lib/lang/validator.js#L85
'path' is already declared in the upper scope on line 3 column 7
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L6
'consoleLogStub' is assigned a value but never used
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L25
Use toHaveLength() instead
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L28
Test title is used multiple times in the same describe block
build (18.x, ubuntu-latest)
The job was canceled because "_16_x_ubuntu-latest" failed.
build (18.x, ubuntu-latest): lib/lang/validator.js#L34
'path' is already declared in the upper scope on line 3 column 7
build (18.x, ubuntu-latest): lib/lang/validator.js#L39
Do not access Object.prototype method 'hasOwnProperty' from target object
build (18.x, ubuntu-latest): lib/lang/validator.js#L57
Duplicate name 'validate'
build (18.x, ubuntu-latest): lib/lang/validator.js#L59
'path' is already declared in the upper scope on line 3 column 7
build (18.x, ubuntu-latest): lib/lang/validator.js#L60
for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array
build (18.x, ubuntu-latest): lib/lang/validator.js#L60
The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype
build (18.x, ubuntu-latest): lib/lang/validator.js#L85
'path' is already declared in the upper scope on line 3 column 7
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L6
'consoleLogStub' is assigned a value but never used
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L25
Use toHaveLength() instead
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L28
Test title is used multiple times in the same describe block
build (18.x, windows-2019)
The job was canceled because "_16_x_ubuntu-latest" failed.
build (18.x, windows-2019)
The operation was canceled.
build (16.x, windows-2019)
The job was canceled because "_16_x_ubuntu-latest" failed.
build (16.x, windows-2019)
The operation was canceled.
build (18.x, macos-latest)
The job was canceled because "_16_x_ubuntu-latest" failed.
build (18.x, macos-latest)
The operation was canceled.
build (16.x, macos-latest)
The job was canceled because "_16_x_ubuntu-latest" failed.
build (16.x, macos-latest)
The operation was canceled.
build (16.x, ubuntu-latest): lib/ScssValidator.js#L85
Unexpected unnamed function
build (16.x, ubuntu-latest): lib/lang/validator.js#L6
Replace `mi` with `im`
build (16.x, ubuntu-latest): lib/lang/validator.js#L29
Replace `'Warning:·Your·theme·has·some·missing·translations·used·in·the·theme:'.yellow` with `⏎················'Warning:·Your·theme·has·some·missing·translations·used·in·the·theme:'.yellow,⏎············`
build (16.x, ubuntu-latest): lib/lang/validator.js#L45
Delete `··········`
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L18
Replace `⏎················process.cwd(),⏎················'test/_mocks/themes/valid',⏎············` with `process.cwd(),·'test/_mocks/themes/valid'`
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L29
Replace `⏎················process.cwd(),⏎················'test/_mocks/themes/valid',⏎············` with `process.cwd(),·'test/_mocks/themes/valid'`
build (16.x, ubuntu-latest): lib/lang/validator.spec.js#L34
Replace `"en"` with `'en'`
build (18.x, ubuntu-latest): lib/ScssValidator.js#L85
Unexpected unnamed function
build (18.x, ubuntu-latest): lib/lang/validator.js#L6
Replace `mi` with `im`
build (18.x, ubuntu-latest): lib/lang/validator.js#L29
Replace `'Warning:·Your·theme·has·some·missing·translations·used·in·the·theme:'.yellow` with `⏎················'Warning:·Your·theme·has·some·missing·translations·used·in·the·theme:'.yellow,⏎············`
build (18.x, ubuntu-latest): lib/lang/validator.js#L45
Delete `··········`
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L18
Replace `⏎················process.cwd(),⏎················'test/_mocks/themes/valid',⏎············` with `process.cwd(),·'test/_mocks/themes/valid'`
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L29
Replace `⏎················process.cwd(),⏎················'test/_mocks/themes/valid',⏎············` with `process.cwd(),·'test/_mocks/themes/valid'`
build (18.x, ubuntu-latest): lib/lang/validator.spec.js#L34
Replace `"en"` with `'en'`