Matt L comments

Results 6 comments of


                                            Matt L

Optional identity columns

We would love this change. It also would solve https://github.com/rsim/oracle-enhanced/issues/2049.

Optional identity columns

@yahonda Sorry to ping directly, but I wonder if you have any reservations about this PR or any guidance on things to improve so it can be accepted. I would...

Use longer identifier for `DatabaseLimits` when using Oracle 12.2 or higher

Is there any work that needs to be completed on this? If so, I'd be happy to help. I am eager to see this in a release!

Add Faker::Date.weekday_between

@stefannibrasil I am still interested, apologies for letting it go stale. It's back on my radar.

Make insert return an array, and include the id_value if super returns nil (12)

Hi @andynu, thank you for your PRs. I had a few questions. 1. What is the intention of the 12 separate MRs? Do they provide different changes or are they...

ActiveRecord: Fix preloading when using a double as foreign_key

I feel similarly over on https://github.com/rails/rails/pull/43402, which also happens to fix something with preloading. I hope you can get eyes on yours soon!