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

use SQLite3::Exception consistently #490

Merged
merged 2 commits into from Jan 28, 2024

Conversation

flavorjones
Copy link
Member

@flavorjones flavorjones commented Jan 27, 2024

use SQLite3::Exception classes consistently

  • Pragmas methods were raising StandardError
  • Database#execute_batch2 was raising RuntimeError
  • Database#load_extension was raising RuntimeError
  • the fallback for unknown errors was RuntimeError

Fixes #302

@flavorjones flavorjones force-pushed the flavorjones-exec-batch-exceptions branch 2 times, most recently from 16f3581 to c5f8f50 Compare January 27, 2024 16:59
- Pragmas methods were raising StandardError
- Database#execute_batch2 was raising RuntimeError
- Database#load_extension was raising RuntimeError
- the fallback for unknown errors was RuntimeError
- use SQLite3::Exception classes consistently
@flavorjones flavorjones force-pushed the flavorjones-exec-batch-exceptions branch from c5f8f50 to d15fdc6 Compare January 27, 2024 17:01
@tenderlove tenderlove merged commit 75c0f5f into main Jan 28, 2024
101 checks passed
@tenderlove tenderlove deleted the flavorjones-exec-batch-exceptions branch January 28, 2024 16:54
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.

Fix database#execute_batch2 errors
2 participants