Implement ESLint rules based on TypeScript/JavaScript patterns analysis with test exceptions and update Copilot instructions #785
kraenhansenAnnotations
10 errors, 10 warnings, and 9 notices
|
Lint:
eslint.config.js#L6
`typescript-eslint` import should occur after import of `eslint-plugin-unused-imports`
|
|
Lint:
eslint.config.js#L4
`globals` import should occur after import of `eslint-plugin-unused-imports`
|
|
Lint:
eslint.config.js#L3
`eslint/config` import should occur after import of `@eslint/js`
|
|
Lint:
apps/test-app/index.ts#L1
There should be at least one empty line between import groups
|
|
Lint:
apps/test-app/App.tsx#L75
`import()` type annotations are forbidden
|
|
Lint:
apps/test-app/App.tsx#L12
`@react-native-node-api/node-tests` import should occur before import of `react`
|
|
Lint:
apps/test-app/App.tsx#L11
`@react-native-node-api/node-addon-examples` import should occur before import of `react`
|
|
Lint:
apps/test-app/App.tsx#L4
`mocha-remote-react-native` import should occur before import of `react`
|
|
Lint:
apps/test-app/App.tsx#L4
There should be no empty line within import group
|
|
Lint:
apps/test-app/App.tsx#L2
There should be no empty line within import group
|
|
Lint:
packages/cmake-rn/src/platforms/android.ts#L106
Unnecessary conditional, comparison is always true, since `"current-development" === "current-development"` is true
|
|
Lint:
packages/cmake-rn/src/cli.ts#L316
Unexpected object value in conditional. The condition is always true
|
|
Lint:
packages/cmake-rn/src/cli.ts#L316
Unnecessary conditional, value is always falsy
|
|
Lint:
packages/cmake-rn/src/cli.ts#L223
Unexpected value in conditional. A boolean expression is required
|
|
Lint:
packages/cmake-rn/src/cli.ts#L206
Unnecessary conditional, comparison is always true, since `0 === 0` is true
|
|
Lint:
packages/cmake-rn/src/cli.ts#L198
Unexpected value in conditional. A boolean expression is required
|
|
Lint:
packages/cmake-rn/src/cli.ts#L189
Unexpected any value in conditional. An explicit comparison or type conversion is required
|
|
Lint:
packages/cmake-rn/src/cli.ts#L48
Unexpected nullable string value in conditional. Please handle the nullish/empty cases explicitly
|
|
Lint:
packages/cli-utils/src/errors.ts#L31
Unexpected any value in conditional. An explicit comparison or type conversion is required
|
|
Lint:
packages/cli-utils/src/actions.ts#L37
Unexpected nullable string value in conditional. Please handle the nullish/empty cases explicitly
|
|
Unit tests (ubuntu-latest)
Total Tests: 20
Suites 📂: 8
Passed ✅: 20
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 698.235ms
|
|
Unit tests (ubuntu-latest)
Total Tests: 3
Suites 📂: 2
Passed ✅: 3
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 534.988ms
|
|
Unit tests (ubuntu-latest)
Total Tests: 49
Suites 📂: 24
Passed ✅: 48
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 1
Todo 📝: 0
Duration 🕐: 126234.683ms
|
|
Unit tests (macos-latest)
Total Tests: 20
Suites 📂: 8
Passed ✅: 20
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 458.332ms
|
|
Unit tests (macos-latest)
Total Tests: 3
Suites 📂: 2
Passed ✅: 3
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 613.494ms
|
|
Unit tests (macos-latest)
Total Tests: 54
Suites 📂: 28
Passed ✅: 54
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 111797.189ms
|
|
Unit tests (windows-latest)
Total Tests: 20
Suites 📂: 8
Passed ✅: 20
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 953.979ms
|
|
Unit tests (windows-latest)
Total Tests: 3
Suites 📂: 2
Passed ✅: 3
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 776.778ms
|
|
Unit tests (windows-latest)
Total Tests: 49
Suites 📂: 24
Passed ✅: 46
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 3
Todo 📝: 0
Duration 🕐: 189421.986ms
|