Skip to content

fix(active_record): support rails 8 connection pool API#274

Merged
matthv merged 8 commits intomainfrom
fix/active-record-connection-pool
Mar 2, 2026
Merged

fix(active_record): support rails 8 connection pool API#274
matthv merged 8 commits intomainfrom
fix/active-record-connection-pool

Conversation

@matthv
Copy link
Member

@matthv matthv commented Feb 27, 2026

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

@qltysh
Copy link

qltysh bot commented Feb 27, 2026

1 new issue

Tool Category Rule Count
qlty Structure Function with high complexity (count = 6): init_native_query_pools 1

db_config,
owner_name: "ForestAdminNativeQuery::#{spec_name}"
)
end
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function with high complexity (count = 6): init_native_query_pools [qlty:function-complexity]

@matthv matthv merged commit f02d792 into main Mar 2, 2026
35 checks passed
@matthv matthv deleted the fix/active-record-connection-pool branch March 2, 2026 13:42
forest-bot added a commit that referenced this pull request Mar 2, 2026
## [1.24.9](v1.24.8...v1.24.9) (2026-03-02)

### Bug Fixes

* **active_record:** support rails 8 connection pool API ([#274](#274)) ([f02d792](f02d792))
@forest-bot
Copy link
Member

🎉 This PR is included in version 1.24.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants