Skip to content

refactor: code simplification #731

refactor: code simplification

refactor: code simplification #731

Triggered via pull request November 8, 2024 17:22
@vicbvicb
synchronize #627
misc/refactor
Status Failure
Total duration 48s
Artifacts

check.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

3 errors, 10 warnings, and 10 notices
validate
tests-unit#test: command (/home/runner/work/opennextjs-aws/opennextjs-aws/packages/tests-unit) pnpm run test exited (1)
packages/tests-unit/tests/http/utils.test.ts > parseCookies > returns an empty list if cookies is emptyish: packages/tests-unit/tests/http/utils.test.ts#L5
AssertionError: expected [] to be [] // Object.is equality If it should pass with deep equality, replace "toBe" with "toStrictEqual" Expected: [] Received: serializes to the same string Compared values have no visual difference. ❯ packages/tests-unit/tests/http/utils.test.ts:5:30
validate
Process completed with exit code 1.
lint/style/useSelfClosingElements: examples/app-pages-router/app/albums/@modal/(.)[album]/[song]/page.tsx#L19
JSX elements without children should be marked as self-closing. In JSX, it is valid for any element to be self-closing.
lint/style/noUnusedTemplateLiteral: .changeset/aggregate.mjs#L50
Do not use template literals if interpolation and special-character handling are not needed.
lint/style/useSelfClosingElements: examples/app-pages-router/app/albums/[album]/[song]/page.tsx#L17
JSX elements without children should be marked as self-closing. In JSX, it is valid for any element to be self-closing.
lint/correctness/useJsxKeyInIterable: examples/app-pages-router/app/albums/page.tsx#L9
Missing key property for this element in iterable.
lint/style/noUselessElse: examples/app-pages-router/middleware.ts#L14
This else clause can be omitted because previous branches break early.
lint/style/noUselessElse: examples/app-pages-router/middleware.ts#L18
This else clause can be omitted because previous branches break early.
lint/style/noUselessElse: examples/app-pages-router/middleware.ts#L25
This else clause can be omitted because previous branches break early.
lint/style/useSelfClosingElements: examples/app-router/app/albums/@modal/(.)[album]/[song]/page.tsx#L19
JSX elements without children should be marked as self-closing. In JSX, it is valid for any element to be self-closing.
lint/performance/noDelete: packages/tests-unit/tests/core/routing/util.test.ts#L528
Avoid the delete operator which can impact performance.
lint/complexity/useLiteralKeys: packages/tests-unit/tests/core/routing/util.test.ts#L528
The computed expression can be simplified without the use of a string literal.
lint/suspicious/noExplicitAny: packages/utils/src/logger.ts#L1
Unexpected any. Specify a different type.
lint/suspicious/noExplicitAny: packages/utils/src/logger.ts#L7
Unexpected any. Specify a different type.
lint/suspicious/noExplicitAny: packages/utils/src/logger.ts#L11
Unexpected any. Specify a different type.
lint/suspicious/noExplicitAny: packages/tests-unit/tests/core/routing/util.test.ts#L29
Unexpected any. Specify a different type.
lint/suspicious/noExplicitAny: packages/tests-unit/tests/core/routing/util.test.ts#L435
Unexpected any. Specify a different type.
lint/suspicious/noExplicitAny: examples/app-router/app/sse/page.tsx#L6
Unexpected any. Specify a different type.