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

Add to options arguments to be passed to the sqlite process #2

Closed
yoavain opened this issue Feb 1, 2022 · 3 comments
Closed

Add to options arguments to be passed to the sqlite process #2

yoavain opened this issue Feb 1, 2022 · 3 comments

Comments

@yoavain
Copy link

yoavain commented Feb 1, 2022

For example: separator can be very useful.
I'm trying to use query instead of get (due to #1) and my data contains the default separator "|"

@WebReflection
Copy link
Owner

as mentioned there:

  • if you need to get data you need to use get or all
  • separator makes no sense in here because the output is -json ... if you don't have the right output is because you are not using SQLite 3.33 or higher

@WebReflection
Copy link
Owner

@yoavain as the #1 bug has been fixed, do you need anything else? I am a bit skeptical at exposing all arguments because I don't have time to fix issues peole might cause by abusing those arguments ... as it is now, everything one needs is there and works out of the box. -bail and -json looks like the only arguments we should care about in this project, thoughts?

@yoavain
Copy link
Author

yoavain commented Feb 2, 2022

Yes, I just saw #1 is fixed, so you're right.
The json API is everything I need.
I guess adding options means more maintenance, and more error prone...

@yoavain yoavain closed this as completed Feb 2, 2022
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

2 participants