Skip to content

fix: update CI workflow for Node.js 24 + fix better-sqlite3 compilation#2

Merged
AxaySharma merged 3 commits into
developfrom
fix/ci-workflow
Jun 4, 2026
Merged

fix: update CI workflow for Node.js 24 + fix better-sqlite3 compilation#2
AxaySharma merged 3 commits into
developfrom
fix/ci-workflow

Conversation

@AxaySharma

Copy link
Copy Markdown
Contributor
  • Pin action versions to latest (checkout@v4.2.2, setup-node@v4.4.0, pnpm@v4)
  • Set FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true env variable
  • Add system deps (python3, make, g++) needed to compile better-sqlite3 on Ubuntu
  • Use pnpm rebuild after install instead of --build-from-source flag
  • Add pnpm cache to speed up future runs

- Pin action versions to latest (checkout@v4.2.2, setup-node@v4.4.0, pnpm@v4)
- Set FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true env variable
- Add system deps (python3, make, g++) needed to compile better-sqlite3 on Ubuntu
- Use pnpm rebuild after install instead of --build-from-source flag
- Add pnpm cache to speed up future runs
.join('|') returns string not array — fix type annotation
resolves TS2322 and TS2345 strict mode errors
@AxaySharma AxaySharma self-assigned this Jun 4, 2026
@AxaySharma

Copy link
Copy Markdown
Contributor Author

Looks good - can be merged.

@AxaySharma
AxaySharma merged commit d1b0020 into develop Jun 4, 2026
2 checks passed
@AxaySharma
AxaySharma deleted the fix/ci-workflow branch June 4, 2026 20:42
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