Skip to content

Conversation

TerrenceSullivanSC
Copy link

@TerrenceSullivanSC TerrenceSullivanSC commented Mar 1, 2025

Allow defaults to be used in querying the SQL mode

Fixes #278

@TerrenceSullivanSC TerrenceSullivanSC requested a review from a team as a code owner March 1, 2025 19:04
@swissspidy
Copy link
Member

Thanks for yor PR, but what issue does this fix? Tests are failing right now

@swissspidy swissspidy changed the title Only unset defaults before running command. Allow defaults to be used… Only unset defaults before running command. Mar 2, 2025
@kraftner
Copy link

@swissspidy Is there a way you can trigger a re-run of the tests? The logs got deleted by now so it is hard to debug the PR which looks sane at first glance on a code level.

@swissspidy swissspidy closed this Oct 14, 2025
@swissspidy swissspidy reopened this Oct 14, 2025
@swissspidy
Copy link
Member

They're re-running now 👍

Copy link

codecov bot commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

get_current_sql_modes() fails if TLS is being used.

4 participants