On should accept a block #77

rzane · 2017-09-11T16:01:49Z

No description provided.

coveralls · 2017-09-11T16:06:40Z

Coverage decreased (-0.1%) to 98.217% when pulling fc70e9d on master-with-updates into c399008 on master.

coveralls · 2017-09-11T19:23:31Z

Coverage decreased (-0.1%) to 98.217% when pulling debb49d on master-with-updates into c399008 on master.

coveralls · 2017-10-15T16:21:35Z

Coverage decreased (-0.1%) to 98.22% when pulling 0c164ca on master-with-updates into c399008 on master.

Ray Zane added 2 commits September 9, 2017 12:15

Add reordering

e2e2a79

Dont need to describe a class for these integration specs

8e449c3

rzane changed the title ~~Join scopes and #reordering~~ Join scopes Sep 11, 2017

Ray Zane added 6 commits September 11, 2017 14:08

Joining a scope kind of works, but it's ugly

0372694

Check type of node before checking

3cc73db

Avoid a monkey-patch in Arel

a7d3c17

Allow using #on with a block

299216b

We dont need to add the alias name as a method

caec74e

No need for custom exception

4e184d8

rzane force-pushed the master-with-updates branch from fc70e9d to 4e184d8 Compare September 11, 2017 18:08

Repository owner deleted a comment from coveralls Sep 11, 2017

This is a better test

debb49d

rzane force-pushed the master-with-updates branch from 5a19635 to debb49d Compare September 11, 2017 19:15

Repository owner deleted a comment from coveralls Sep 11, 2017

rzane mentioned this pull request Sep 17, 2017

Aliases don't work in 'on' and 'select' conditions #81

Closed

Joining scopes is not quite ready

0c164ca

rzane changed the title ~~Join scopes~~ On should accept a block Oct 15, 2017

rzane merged commit a3b7399 into master Oct 15, 2017

rzane deleted the master-with-updates branch October 15, 2017 16:33

Provide feedback