Make SelectManager#distinct chainable #117

Closed
wants to merge 1 commit into
from

6 participants

@nilclass

Does anyone know any reason why this would be bad? If not, please merge :)

@josephbridgwaterrowe

Hi @nilclass,

Could you provide an example of using distinct chained and non-chained please?

Thanks!

@nilclass

Using distinct chains I was doing here: https://github.com/nilclass/path_finder/blob/master/lib/path_finder/query_builder.rb#L147, using non-chained I need not provide (and do not have) any example, as that is the behavior I'm criticizing by this pull request. ok?

@beedub

👍

@mbursi

+1

@tamird

Obsolete, this was added in acdd8e4. good to close

@matthewd matthewd closed this Sep 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment