Skip to content

Conversation

@G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Dec 6, 2025

Summary

connection_pool v3+ requires keyword arguments instead of positional args, but also drops support for Ruby 3.2 so this has us check the version being used and do the right thing accordingly

Other Information

Resolves #1366

Pull Request checklist

  • Add/update test to cover these changes
  • Update documentation
  • Update CHANGELOG file

@G-Rath G-Rath force-pushed the connection_pool_v3 branch 4 times, most recently from d2dab3b to 541a320 Compare December 6, 2025 21:28
@G-Rath G-Rath force-pushed the connection_pool_v3 branch from 541a320 to 4801998 Compare December 6, 2025 21:31
@G-Rath G-Rath marked this pull request as ready for review December 6, 2025 21:55
@mhfs
Copy link

mhfs commented Dec 8, 2025

Thanks @G-Rath. Came here for the same issue and was happy to see your PR.

@G-Rath G-Rath force-pushed the connection_pool_v3 branch from d8b64bf to e9348c6 Compare December 8, 2025 18:20
@izn
Copy link

izn commented Dec 9, 2025

Any updates on when this can be merged?

G-Rath added a commit to ackama/rails-template that referenced this pull request Dec 11, 2025
Currently `react-rails` does not work with `connection_pool` v3, so we
need to lock to v2 until
reactjs/react-rails#1367 is released
@justin808 justin808 merged commit a7d4678 into reactjs:main Dec 12, 2025
27 checks passed
justin808 pushed a commit that referenced this pull request Dec 12, 2025
* test: add appraisal gemfile for `connection_pool` v2

* test: add coverage for `connection_pool` v3

* fix: support `connection_pool` v3

* docs: add changelog entry

* refactor simplify fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incompatible with v3 of connection_pool

4 participants