-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Update drizzle config to be up to date with the latest version #1881
fix: Update drizzle config to be up to date with the latest version #1881
Conversation
🦋 Changeset detectedLatest commit: 7e765e7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@gcjbr is attempting to deploy a commit to the t3-oss Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you bump the versions in dependencyVersionMap as well just so that it will always pull ^0.21 no matter what package manager? Thanks!
Done |
CI failing |
I've bumped the dependencyVersionMap. I see the linter is failing, it's failing locally on the main branch as well, on the same places it's failing here. Weird 👁️ |
okay it was the big lockfile diff that was causing the issues i reverted the lock file changes and resynced it and thing's are back to normal |
idk what pnpm version you're on but you had caused like 6k lines of diff in the lockfile which i reverted b0e6dfe |
pnpm -v Nice catch, thanks! I'll fix the other test |
@juliusmarminge , done! Can you tell me your PNPM version, btw? Just so I don't create a mess in the future |
I use corepack so it matches whatever is in package.json#packageManager |
…3-oss#1881) Co-authored-by: juliusmarminge <[email protected]>
Closes #
✅ Checklist
Changelog
_ drizzle-config-sqlite.ts, drizzle-config-postgres.ts and drizzle-config-mysql.ts were changed to adhere to Drizzle's format
Screenshots
While running Drizzle Studio we are always asked to update it to the latest version, or it won't show the UI. After tha last update, it's finding an error with the current template configs.
This fixes it.
💯