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

fix: numerical values in options as int #18

Merged
merged 3 commits into from
Nov 28, 2023

Conversation

estahn
Copy link
Contributor

@estahn estahn commented Aug 15, 2023

We require connect_timeout for MySQL in order to fail if it can not connect within a certain period. Thus avoiding Lambda to execute the full allotted time.

The fix for dj-database-url (jazzband/dj-database-url#222) was accepted, merged and released in 2.1.0.

I have updated the version requirement and added details to the test-case.

@estahn
Copy link
Contributor Author

estahn commented Aug 15, 2023

@erhosen Let me know if there is anything further required to get this merged and released.

@woodsae
Copy link

woodsae commented Aug 15, 2023

Brilliant! This will solve my issue as well

@estahn
Copy link
Contributor Author

estahn commented Aug 18, 2023

@erhosen Any update on this?

@andrey-lepekhin andrey-lepekhin self-assigned this Nov 27, 2023
@andrey-lepekhin
Copy link
Collaborator

@estahn Thank you! Merging.
Really sorry for the delay, there was a problem with GitHub notifications delivery — they where not coming through at all :(

@andrey-lepekhin andrey-lepekhin merged commit 861c165 into erhosen-libs:master Nov 28, 2023
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

Successfully merging this pull request may close these issues.

3 participants