-
Notifications
You must be signed in to change notification settings - Fork 9
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 types to query context on PgError #42
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.
LGTM except failing CI
Alright. The issue was something in the Lodash types was unable to resolve in the tests given the test tsconfig. I just set it to strict, fixed the issues, and the lodash compile issue went away. should be good to go if the test changes are approved. |
@dynajoe Do you want to merge and release this? |
Wanted to follow up here since it's been a little while! Would be fun to have this fix in. |
@dynajoe Can you release this change? |
Hi Stephen |
👋 Do you know how to kick off releases? |
Fixes #40