Shinigami comments

Results 864 comments of


                                            Shinigami

LoremFlickr down for good?

**Team Decision**: - We will not remove functions like `urlLoremFlickr` unless the external service was fully taken down - If the external service is unstable, partly down or does not...

BUG: Does not support `prettier.config.ts`, recently added in Prettier `v3.5.0`

> using `prettier.config.js` with `// @ts-check` and jsdoc types That's not a workaround 🤦 Please read the title of the issue...

feat: expose createMigrationBuilder to allow using this lib as sql builder

Sorry for the late response, but life... you know What is the purpose of this PR? You wrote about not having file accesses. Potentially we need to go another route...

feat: expose createMigrationBuilder to allow using this lib as sql builder

I don't feel safe with that approach. Even unstable would be used and required some day and depend on. I personally don't like to expose `MigrationBuilderImpl` because it could be...

Add a way to run `.ts` files in Programmatic

Would you like to test out this? https://github.com/salsita/node-pg-migrate/pull/1187#issuecomment-2158871288

Add a way to run `.ts` files in Programmatic

Are you able to create a minimal reproducible repository I can checkout?

Add a way to run `.ts` files in Programmatic

thx, just to set expectation, I will need to consume free-time for that, hopefully I find some time tomorrow

Add a way to run `.ts` files in Programmatic

looks like I did not found time "tomorrow" 😬 ... my current plan is to go full esm only support and potentially even drop NodeJS v18 support, so I can...

Add a way to run `.ts` files in Programmatic

@brenoepics is working on https://github.com/salsita/node-pg-migrate/pull/1460 This might fix this issue

Support ALTER INDEX expressions (or the same thing in ALTER TABLE)

Yeah, I see Are you up for making a PR? Start at least with the more simple one `src/operations/indexes/renameIndex.ts`