fractaledmind / acidic_job

Elegant, resilient, durable workflows for Rails apps
https://fractaledmind.github.io/acidic_job/
MIT License
492 stars 10 forks source link

Remove lambda from default value in migration #97

Closed ahacop closed 1 week ago

ahacop commented 2 months ago

This resolves an issue with the ActiveRecord SQLite adapter, which raises an error when the lambda contains values that don't respond to match?.

https://github.com/rails/rails/blob/e1d58cfd05ae1cc0bfc1006b7ce973a7730831df/activerecord/lib/active_record/connection_adapters/sqlite3/quoting.rb#L89

Fixes https://github.com/fractaledmind/acidic_job/issues/95

Sorry about the noise around the multiple PRs. I moved the change to a different branch on my fork and it permanently closed the other PR.

ahacop commented 1 week ago

Closing this PR as alpha-v1.0 introduces significant changes that make it irrelevant.