Skip to content

refactor: rewrite migration cli with tiny-cli#13

Merged
hi-ogawa merged 17 commits intomainfrom
feat-generalized-migrator
Jul 11, 2023
Merged

refactor: rewrite migration cli with tiny-cli#13
hi-ogawa merged 17 commits intomainfrom
feat-generalized-migrator

Conversation

@hi-ogawa
Copy link
Copy Markdown
Owner

@hi-ogawa hi-ogawa commented Jul 10, 2023

It seems it's easy to replace kysely's migration system with somewhat more generalized version, which then allows easier integration with D1.
Probably I'll move this to independent package in https://github.com/hi-ogawa/js-utils/

todo

  • test cli on remote D1
    • drop all tables including old kysely migration tables
      • npx wrangler d1 execute demo --command 'drop table counter; drop table kysely_migration; drop table kysely_migration_lock'
      • just recreated a whole D1 also with --experimental-backend
    • run pnpm migration-produciton up
  • publish independent pacakge
  • remove kysely

@hi-ogawa hi-ogawa marked this pull request as ready for review July 10, 2023 13:25
@hi-ogawa hi-ogawa force-pushed the feat-generalized-migrator branch from 10251fa to 62a3f26 Compare July 11, 2023 00:10
@hi-ogawa hi-ogawa mentioned this pull request Jul 11, 2023
hi-ogawa added a commit to hi-ogawa/js-utils that referenced this pull request Jul 11, 2023
hi-ogawa added a commit to hi-ogawa/js-utils that referenced this pull request Jul 11, 2023
hi-ogawa added a commit to hi-ogawa/js-utils that referenced this pull request Jul 11, 2023
hi-ogawa added a commit to hi-ogawa/js-utils that referenced this pull request Jul 11, 2023
hi-ogawa added a commit to hi-ogawa/js-utils that referenced this pull request Jul 11, 2023
@hi-ogawa hi-ogawa changed the title refactor: simplify migration cli refactor: rewrite migration cli with tiny-cli Jul 11, 2023
@hi-ogawa hi-ogawa merged commit 3c90331 into main Jul 11, 2023
@hi-ogawa hi-ogawa deleted the feat-generalized-migrator branch July 11, 2023 13:34
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