-
Notifications
You must be signed in to change notification settings - Fork 116
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
1.0.0 #9
1.0.0 #9
Conversation
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.
Awesome job Lufy! This package is looking really good now, and this was a big help.
I add a bit of changes in the comments, a lot of the changes are just adding a max
parameter for queries that return a large amount of data.
Once these fixes can handled, I'm cool with deploying this to npm as the official v1.0.0
Co-authored-by: jiro-ono <70732637+jiro-ono@users.noreply.github.com>
Co-authored-by: jiro-ono <70732637+jiro-ono@users.noreply.github.com>
Co-authored-by: jiro-ono <70732637+jiro-ono@users.noreply.github.com>
LGTM! I'm merging into 1.0.0, and will deploy it. |
No description provided.