Implement ESLint rules based on TypeScript/JavaScript patterns analysis with test exceptions and update Copilot instructions #782
Triggered via pull request
October 26, 2025 22:16
kraenhansen
Status
Cancelled
Total duration
5m 27s
Artifacts
–
Annotations
16 errors, 10 warnings, and 3 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
|
|
Unit tests (ubuntu-latest)
Canceling since a higher priority waiting request for Check-refs/pull/290/merge exists
|
|
Unit tests (windows-latest)
Canceling since a higher priority waiting request for Check-refs/pull/290/merge exists
|
|
Unit tests (windows-latest)
The operation was canceled.
|
|
Unit tests (macos-latest)
Canceling since a higher priority waiting request for Check-refs/pull/290/merge exists
|
|
Unit tests (macos-latest)
The operation was canceled.
|
|
Check
Canceling since a higher priority waiting request for Check-refs/pull/290/merge exists
|
|
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 🕐: 640.664ms
|
|
Unit tests (ubuntu-latest)
Total Tests: 3
Suites 📂: 2
Passed ✅: 3
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 0
Todo 📝: 0
Duration 🕐: 525.841ms
|
|
Unit tests (ubuntu-latest)
Total Tests: 49
Suites 📂: 24
Passed ✅: 48
Failed ❌: 0
Canceled 🚫: 0
Skipped ⏭️: 1
Todo 📝: 0
Duration 🕐: 154081.007ms
|