Skip to content

Conversation

@paulhenri-l
Copy link
Contributor

This PR introduces two main changes:

  • Replaces the custom PostgreSQL migration script with Drizzle migrator.
  • Switches to a dedicated PostgreSQL schema to isolate workflow tables from user tables.

These changes add support for future schema update and ensure we don’t interfere with user-defined tables.

A new DEV_NOTES.md file has been added to explain how to generate new migrations. Ideally, this should be automated as part of the build process, but I’m not yet familiar with workflow’s build system. Any guidance on this would be appreciated.

Signed-off-by: paulhenri-l <[email protected]>
Signed-off-by: paulhenri-l <[email protected]>
@changeset-bot
Copy link

changeset-bot bot commented Nov 12, 2025

🦋 Changeset detected

Latest commit: 382a0ca

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@workflow/world-postgres Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Contributor

vercel bot commented Nov 12, 2025

@paulhenri-l is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​drizzle-orm@​0.31.4 ⏵ 0.44.79810088 +199100
Updatednpm/​drizzle-kit@​0.22.8 ⏵ 0.31.69910096 +199 +2100

View full report

@@ -0,0 +1,7 @@
# Generate migrations
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file will get missed. let's just add it into the README of the repo itself.

Typically claude, etc. will also read the README when trying to make changes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I will add this change in the next PR

@pranaygp pranaygp merged commit 5790cb2 into vercel:main Nov 12, 2025
19 of 36 checks passed
@paulhenri-l paulhenri-l deleted the world-postgres-migrator branch November 12, 2025 17:15
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