Skip to content

[codex] Add tests and lint CI#4

Merged
0xSwego merged 1 commit into
codex/add-bounds-selectionfrom
codex/tests-lint-ci
Jun 10, 2026
Merged

[codex] Add tests and lint CI#4
0xSwego merged 1 commit into
codex/add-bounds-selectionfrom
codex/tests-lint-ci

Conversation

@Faolain

@Faolain Faolain commented Jun 9, 2026

Copy link
Copy Markdown

Summary

Adds a dedicated GitHub Actions workflow for local JS quality checks, with separate lint and Vitest steps.

Also adds ESLint flat config and fixes the existing explicit any usage so @typescript-eslint/no-explicit-any can run as an active lint rule instead of being suppressed.

Details

  • Adds lint and test:ci package scripts.
  • Adds ESLint dependencies and eslint.config.js.
  • Adds .github/workflows/tests-and-lint.yml for all pull requests and pushes to main.
  • Replaces explicit any usage with concrete object types, unknown, jaxray selection types, and typed STAC metadata accessors.
  • Removes a focused Haversine test marker and fixes stale distance assertions that CI exposed.
  • Cleans up minor lint issues discovered while enabling the gate.

Validation

  • corepack yarn lint
  • corepack yarn build
  • CI=1 corepack yarn test:ci

@0xSwego 0xSwego force-pushed the codex/tests-lint-ci branch from 68a1171 to 115a8ed Compare June 9, 2026 15:55
@0xSwego 0xSwego marked this pull request as ready for review June 9, 2026 15:57
@0xSwego 0xSwego force-pushed the codex/tests-lint-ci branch from 115a8ed to 8d79171 Compare June 9, 2026 16:01
@Faolain Faolain marked this pull request as draft June 9, 2026 16:02
@0xSwego 0xSwego marked this pull request as ready for review June 10, 2026 09:50
@0xSwego 0xSwego merged commit 8d79171 into codex/add-bounds-selection Jun 10, 2026
1 check passed
@0xSwego 0xSwego deleted the codex/tests-lint-ci branch June 10, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants