add support for rails v8#521
Conversation
jamesoneill997
requested review from
sarahraqueld and
smaboshe
as code owners
|
Hey @smaboshe // @sarahraqueld , Any ideas on a timeline for getting a review on this? I'm more than happy to make any requested changes to get this one working and over the line. |
neilvcarvalho
left a comment
neilvcarvalho
left a comment
There was a problem hiding this comment.
Thanks for your contribution!
Just adding Rails 8.0 to the test matrix won't solve the issues you're having. It could surface them with broken specs, but looks like they're all passing. Would you mind creating an issue describing what you were facing and what workarounds you did, if any?
NEWS.md
Outdated
|
|
||
| # NEWS | ||
|
|
||
| ## 6.4.5 (December 6, 2024) |
There was a problem hiding this comment.
| ## 6.4.5 (December 6, 2024) |
We're going to add the version number and release date when this is released. In the meantime, we just add to the top as unreleased.
Gemfile.lock
Outdated
| remote: . | ||
| specs: | ||
| factory_bot_rails (6.4.4) | ||
| factory_bot_rails (6.4.5) |
There was a problem hiding this comment.
| factory_bot_rails (6.4.5) | |
| factory_bot_rails (6.4.4) |
|
Thanks for the review @neilvcarvalho ! Yeah I was speculatively hoping that this might nudge the issue we are seeing in the right direction, but I don't see it resolving it. I'm just working through some backlog atm, so bear with me, it might be a few days before I get a chance to recreate and make an issue. From my recollection, I was noticing some validation errors were missing when bumping to rails 8, and I had traced it back to this gem. I'll get a proper issue made up in the coming days and add as much detail as I can. Will also tidy this up as per your comments. Thanks again! |
2 participants
Noticed some issues when trying to integrate v6.4.4 with rails v8, hoping that adding support here should resolve things.
Haven't contributed to this repo before, so please feel free to let me know if I've missed anything.