Skip to content

feat: update to typeorm 1.0#80

Closed
maxdietrich-spiribo wants to merge 3 commits into
apache:masterfrom
maxdietrich-spiribo:master
Closed

feat: update to typeorm 1.0#80
maxdietrich-spiribo wants to merge 3 commits into
apache:masterfrom
maxdietrich-spiribo:master

Conversation

@maxdietrich-spiribo

Copy link
Copy Markdown

Hello there 👋

This adds support for the new TypeORM version 1, it fixes the breaking changes and bumps the peer dependencies typeorm expects accordingly. The unit tests pass.

@hsluoyz

hsluoyz commented Jun 17, 2026

Copy link
Copy Markdown
Member

@maxdietrich-spiribo use registry.yarnpkg.com in lock

@maxdietrich-spiribo

Copy link
Copy Markdown
Author

My bad, that was my default locally and overwrote it. Should be fixed now :)

@hsluoyz

hsluoyz commented Jun 17, 2026

Copy link
Copy Markdown
Member

yarn.lock should not be a full-rewrite. Minimize the diff

@maxdietrich-spiribo

Copy link
Copy Markdown
Author

I changed it back to the current master state of yarn.lock and ran yarn install with the old yarn.lock, which I would assume should generate the smallest diff, but that still seems to be close to a complete rewrite.

I wonder if this is caused by a version difference in yarn or something? I am using the default corepack version of yarn 1.22.22 here. I just tried updating to the latest yarn version 4.17.0, but that generates a format that is even more different from the current state.

With what version of yarn was the yarn.lock produced in the current version?

@hsluoyz

hsluoyz commented Jun 18, 2026

Copy link
Copy Markdown
Member

replaced by: a5d88a1

@hsluoyz hsluoyz closed this Jun 18, 2026
@maxdietrich-spiribo

Copy link
Copy Markdown
Author

Thanks for getting this into master :) Is there anything stopping a new release of the package? That would help us in our migration to typeorm 1.0

@hsluoyz

hsluoyz commented Jun 22, 2026

Copy link
Copy Markdown
Member

@maxdietrich-spiribo released: https://www.npmjs.com/package/typeorm-adapter/v/1.10.0

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.

2 participants