Fix for `no implicit conversion of nil into String`

gocardless / statesman

A statesmanlike state machine library.

https://gocardless.com/blog/statesman/

MIT License

1.78k stars 163 forks source link

Fix for `no implicit conversion of nil into String` #427

Closed thom-oman closed 3 years ago

thom-oman commented 3 years ago

As part of https://github.com/gocardless/statesman/pull/421 we changed db_null in the AR adapter to type cast nil. This breaks for Postgres databases that have a nullable most_recent column.

Fixes: https://github.com/gocardless/statesman/issues/424

thom-oman commented 3 years ago

The pending specs are for the old rails master, but rails has switched to 'main' and they're still hanging around, will be fine in future PRs