Thanks for your PR! I like it, except for the checks on enumerable methods (respond_to?(:select), respond_to?(:map)). Since there is also some duplicate code in dup_has_many_association, I'll try to adapt it a little by extracting shared code to methods, making it a little more DRY and simpler to read. Will look into it this week.
Thanks for your PR! I like it, except for the checks on enumerable methods (
respond_to?(:select)
,respond_to?(:map)
). Since there is also some duplicate code indup_has_many_association
, I'll try to adapt it a little by extracting shared code to methods, making it a little more DRY and simpler to read. Will look into it this week.