Skip to content

ci: add GitHub Actions CI and Biome linting - #94

Open
zcaceres wants to merge 4 commits into
mainfrom
ci/add-biome-and-github-actions
Open

ci: add GitHub Actions CI and Biome linting#94
zcaceres wants to merge 4 commits into
mainfrom
ci/add-biome-and-github-actions

Conversation

@zcaceres

@zcaceres zcaceres commented Apr 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds a GitHub Actions CI workflow that runs lint, build, and test on all PRs to main
  • Adds Biome for linting and formatting with recommended rules
  • Reformats codebase to Biome defaults (tabs, double quotes, node: protocol imports)
  • Updates branch protection to require the ci check to pass before merging

Test plan

  • CI workflow triggers on this PR and passes lint/build/test
  • Verify bun run lint, bun run build, and bun test all pass locally

zcaceres added 4 commits April 2, 2026 13:47
- Add CI workflow that runs lint, build, and test on PRs to main
- Add Biome for linting/formatting with recommended rules
- Auto-fix all formatting to Biome defaults (tabs, double quotes)
- Fix lint issues: node: protocol imports, explicit types, non-null assertions
- Exclude sample-data and markdownify/ from linting
- Update branch protection to require CI checks
The markitdown Python process emits warnings to stderr (onnxruntime,
pydub/ffmpeg) that don't indicate failure. execFileAsync already throws
on non-zero exit codes, so checking stderr is redundant and causes
false test failures in CI.
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.

1 participant