Skip to content

fix(typeorm-codegen): avoid unused Column imports - #553

Open
Boulea7 wants to merge 1 commit into
subsquid:masterfrom
Boulea7:fix/typeorm-codegen-column-import-547
Open

fix(typeorm-codegen): avoid unused Column imports#553
Boulea7 wants to merge 1 commit into
subsquid:masterfrom
Boulea7:fix/typeorm-codegen-column-import-547

Conversation

@Boulea7

@Boulea7 Boulea7 commented Aug 11, 2026

Copy link
Copy Markdown

Relates to #547.

Summary

  • Avoid importing Column into scalar-only generated TypeORM models.
  • Preserve Column imports for enum and JSONB-backed generated fields.

Validation

  • node ../../common/scripts/install-run-rushx.js test -- codegen.test.ts — 49 passed.
  • node ../../common/scripts/install-run-rushx.js test — 49 passed.
  • node common/scripts/install-run-rush.js build --to @subsquid/typeorm-codegen — passed.
  • node common/scripts/install-run-rush.js check — passed.
  • node common/scripts/install-run-rush.js change --verify --target-branch origin/release/arrowsquid — passed.

Known limitations

  • Repository-wide and scoped Biome checks report existing diagnostics in the touched legacy files; no unrelated formatting changes were made.
  • rush test:lfs was not run; the local partial clone initially omitted 113 LFS fixtures.
  • The full monorepo unit-test suite was not run.

@Boulea7
Boulea7 marked this pull request as ready for review August 12, 2026 08: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