Skip to content

chore: update CI to Node 22, enable corepack, and fix lockfile#60

Merged
B4nan merged 5 commits intomasterfrom
fix/ci-node22-lockfile
Mar 12, 2026
Merged

chore: update CI to Node 22, enable corepack, and fix lockfile#60
B4nan merged 5 commits intomasterfrom
fix/ci-node22-lockfile

Conversation

@B4nan
Copy link
Copy Markdown
Member

@B4nan B4nan commented Mar 11, 2026

Summary

  • Update CI node-version matrix from [18, 20] to [22] since MikroORM v7 requires Node >= 22.17.0
  • Add corepack enable step to CI workflow for proper Yarn setup
  • Remove the Cache node_modules step (not needed with Yarn PnP)
  • Regenerate yarn.lock which was stale after the v7 upgrade

Test plan

  • Verify CI passes on Node 22 with the updated workflow

🤖 Generated with Claude Code

B4nan and others added 5 commits March 11, 2026 18:56
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The CI was using `corepack prepare yarn@stable` which installs Yarn v4,
but the lockfile was in Yarn v1 format causing "lockfile would have been
modified" errors. Regenerate the lockfile in v4 format with node-modules
linker.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MikroORM v7 requires explicit entity registration. Import all
EntitySchema definitions into the config.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Prevents 'Duplicate table names' error since BaseEntity is not a
concrete entity and should not have its own table.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace EntitySchema definitions with the defineEntity + property
builder API. Use setClass to register custom classes with constructors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@B4nan B4nan merged commit 3ea2a7f into master Mar 12, 2026
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