-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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: updated --dbProvider CI flag to parse configured databases #1865 #1867
Conversation
🦋 Changeset detectedLatest commit: d2e3c89 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 ↗︎
|
@xelacast is attempting to deploy a commit to the t3-oss Team on Vercel. A member of the Team first needs to authorize it. |
GitHub Actions error that I cannot reproduce. The error is here. LibsqlError: URL_SCHEME_NOT_SUPPORTED: The client supports only "libsql:", "wss:", "ws:", "https:", "http:" and "file:" URLs, got "mysql:". For more information, please read https://github.com/libsql/libsql-client-ts#supported-urls I will be trying to solve this but If anyone has any insight on why this is happening it would be much appreciated. EDIT: 05/06/24 The GitHub action e2e error happens once I make the logic for the databaseProvider = cliResults.flags.dbProvider statement on line 200 in cli/src/cli/index.ts. I cannot reproduce this error on my own machine only with the github action e2e.yaml. |
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.
thanks!
ps: sorry for the weird CI issue...
…s#1865 (t3-oss#1867) Co-authored-by: juliusmarminge <julius0216@outlook.com>
Closes #1865
✅ Checklist
Changelog
Inside cli/src/cli/index.ts
Updated CI flags to include dbProvider with a default value of 'sqlite'
Updated default value of --dbProvider
Included an error check for the input of the --dbProvider value
Set cliResults.databaseProvider to a conditional value based on the user input
Included parser for both Drizzle and Prisma
Updated documentation to include CLI flags and examples
Feedback or advice is encouraged 💯
💯