Skip to content
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

CLI feedback could be better when a required argument is missing #25

Closed
tmcw opened this issue Mar 30, 2023 · 2 comments
Closed

CLI feedback could be better when a required argument is missing #25

tmcw opened this issue Mar 30, 2023 · 2 comments

Comments

@tmcw
Copy link
Contributor

tmcw commented Mar 30, 2023

When I try to introspect a database but forget the --migrations argument, the only feedback is "Missing required params" - it'd be great if this CLI listed those required params so one could use it without toggling between --help or consulting the readme.

➜  drizzle-test y drizzle-kit introspect:pg --database=drizzledemo --user=tmcw --password="" --host=0.0.0.0
yarn run v1.22.19
$ /Users/tmcw/v/drizzle-test/node_modules/.bin/drizzle-kit introspect:pg --database=drizzledemo --user=tmcw --password= --host=0.0.0.0
drizzle-kit: v0.17.1
drizzle-orm: v0.23.2

Missing required params
✨  Done in 0.20s.
@m3dragos
Copy link

m3dragos commented May 11, 2023

On a similar note, if the path to the schema is invalid, I simply got the 0 tables; No schema changes, nothing to migrate message. Would be nice to say that the schema file was not found instead.

@AndriiSherman
Copy link
Member

Fixed in drizzle-kit@0.18.1

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

No branches or pull requests

3 participants