Added Arel::Nodes::InfixOperation to Visitors::ToSql#visit_Arel_Nodes…

rails / arel

A Relational Algebra

2.06k stars 390 forks source link

Added Arel::Nodes::InfixOperation to Visitors::ToSql#visit_Arel_Nodes… #477

Closed PedroSena closed 6 years ago

PedroSena commented 7 years ago

PR to address https://github.com/rails/arel/issues/393 .

I'm thinking about adding tests for the other managers as well, I believe this change will benefit them too. For now I'll leave it as it is for verification from @drbawb and the the maintainers too. I can add more tests if the change is acceptable.

matthewd commented 6 years ago

Per #523, Arel development is moving to rails/rails.

If this PR is still relevant, please consider reopening it over there.