Skip to content

🥅 Await promise#27

Open
clickclickonsal wants to merge 15 commits intolinnify:mainfrom
sesolabor:main
Open

🥅 Await promise#27
clickclickonsal wants to merge 15 commits intolinnify:mainfrom
sesolabor:main

Conversation

@clickclickonsal
Copy link
Copy Markdown

getRepository(this.entity).save(entity); returns a promise that was not being awaited. So this caused an issue trying when working postGenerators because the entity was not always guaranteed to be persisted.

TypeORM 0.3.x no longer accepts a raw ID in findOne(). Migrate to
findOne({ where: { id } }) and findOne({ where: { id }, relations })
so the test suite compiles and passes.

Made-with: Cursor
- Downgrade sqlite3 to ^5.1.7 to satisfy typeorm peer dependency
- Add tsconfig.test.json so ts-jest resolves @types/jest globals
- Point jest.config transform at the test tsconfig
- Replace pnpm-lock.yaml with package-lock.json for npm ci in CI

Made-with: Cursor
- Add explicit rootDir for TS6 requirement (TS5011)
- Add types: ["node"] since TS6 no longer auto-includes @types
- Bump lib to es2018 for AsyncGenerator support needed by typeorm

Made-with: Cursor
Made-with: Cursor
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