Using jsk sql commands with a query that would raise an error such as seelect 5 would raise 2 errors, one for the bad query and an UnboundLocalError for output.
Summary of changes made
start with output=None then check if it's still None after executing
Checklist
[x] This PR changes the jishaku module/cog codebase
[ ] These changes add new functionality to the module/cog
[x] These changes fix an issue or bug in the module/cog
[x] I have tested that these changes work on a production bot codebase
[ ] I have tested these changes against the CI/CD test suite
[ ] I have updated the documentation to reflect these changes
[ ] This PR changes the CI/CD test suite
[ ] I have tested my suite changes are well-formed (all tests can be discovered)
[ ] These changes adjust existing test cases
[ ] These changes add new test cases
[ ] This PR changes prose (such as the documentation, README or other Markdown/RST documents)
[ ] I have proofread my changes for grammar and spelling issues
[ ] I have tested that any changes regarding Markdown/RST syntax result in a well formed document
Rationale
Using
jsk sqlcommands with a query that would raise an error such asseelect 5would raise 2 errors, one for the bad query and an UnboundLocalError for output.Summary of changes made
start with output=None then check if it's still None after executing
Checklist