Jean Boussier comments

Results 529 comments of


                                            Jean Boussier

ActiveRecord::NotNullViolation Error When Saving jsonb Column in Rails 7 and Above

Pretty sure regular `store` and `serialize` both require the column to be nulable, and that's why. I haven't double checked but I highly doubt this could be changed on the...

ActiveRecord::NotNullViolation Error When Saving jsonb Column in Rails 7 and Above

I see. I guess it wouldn't be hard to just replace `ActiveRecord::TypedStore::IdentityCoder` by something that's the same but returns an empty hash instead of nil. I don't think it's for...

Make it work with next Rails

I don't think we should bother, but if for some reason we wanted to make it compatible to ease transition, then rather than monkey patch we should use `ActiveRecord.query_transformers`

- PR for fixing the default {} issue #108

I haven't looked at the code in details, but there's tons of CI failures.

HSET error when value to store is empty

It would be easy to do nothing if the argument is empty yes.

Support for Net::HTTPPaymentRequired

Merged https://github.com/rails/activeresource/pull/401

Embed const caches that aren't cref dependent

> I already know RJIT is broken. Actually, it might not be, because it doesn't seem to use const caches that don't have a cref.

Embed const caches that aren't cref dependent

@ruby/yjit would someone have a bit of time to help me update YJIT to handle the new constcache layout? I suspect I need to check for the `HAS_EXT` flag in...

Embed const caches that aren't cref dependent

Reopening, because this might actually be possible using `CAS`.

Embed const caches that aren't cref dependent

@XrXr I think I pushed a new version that work with ractors, but I'm not super accustomed to all the constraints so I'd really appreciate if you could review this....