Skip to content

fix(ocr): update biome.json for v1.9.4 compatibility#10

Merged
willgriffin merged 5 commits intomainfrom
fix/biome-config
Dec 10, 2025
Merged

fix(ocr): update biome.json for v1.9.4 compatibility#10
willgriffin merged 5 commits intomainfrom
fix/biome-config

Conversation

@willgriffin
Copy link
Copy Markdown
Contributor

Summary

  • Update biome.json schema reference from 2.2.6 to 1.9.4 (actual installed version)
  • Rename includes to include (correct key for biome 1.9.x)
  • Remove deprecated useParseIntRadix rule
  • Simplify file include patterns to .ts files

Test plan

  • npm run lint now runs successfully (was failing with config errors)
  • npm run build passes
  • npm run typecheck passes
  • npm test passes (27/27 tests)
  • CI/CD pipeline validates changes

- Change schema reference from 2.2.6 to 1.9.4 (actual installed version)
- Rename 'includes' to 'include' (correct key for biome 1.9.x)
- Remove deprecated 'useParseIntRadix' rule
- Simplify file include patterns to .ts files
- Remove explicit pnpm version from workflows (use packageManager from package.json)
- Rename commitlint.config.js to commitlint.config.mjs (required by wagoid/commitlint-github-action@v6)
- Update on-pull-request workflow to reference .mjs config file
@github-actions
Copy link
Copy Markdown
Contributor

Version Bump Preview

When this PR is merged, @happyvertical/ocr will receive a patch version bump based on your conventional commits.

What happens on merge?

  1. Tests run on main branch
  2. Package is built
  3. Version is bumped automatically
  4. Package is published to GitHub Packages
  5. Git tag is created

No manual intervention needed!

- Add GitHub App token generation to test.yml and changeset-check.yml workflows
- Pass secrets to reusable test.yml workflow from on-pull-request.yml and publish.yml
- Use app token for NODE_AUTH_TOKEN with fallback to GITHUB_TOKEN
- Fixes 403 Forbidden error when fetching @happyvertical packages
- Add owner: happyvertical to create-github-app-token action
- Ensures token has access to packages across all happyvertical repos
@willgriffin willgriffin merged commit 6f42ede into main Dec 10, 2025
1 of 2 checks passed
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