r?ply issues #189

Merged
merged 3 commits into from Jan 6, 2014

2 participants

@krlmlr

Seamless support for both expressions and functions in the r?ply family, maintaining consistency for both side effects and output.

Fixes #158 and #164. Rebased variant of #165.

@hadley
Owner

Could you please merge/rebase again?

@krlmlr

Rebased and force-pushed. I believe there is no need to provide a dedicated implementation for r_ply.

@krlmlr

Test currently failing. Will fix.

@krlmlr

Fixed test, ready to merge.

@hadley hadley merged commit dcfdf26 into hadley:master Jan 6, 2014

1 check passed

Details default The Travis CI build passed
@krlmlr krlmlr deleted the krlmlr:fix/158-rply-expr branch Jan 7, 2014
@krlmlr krlmlr restored the krlmlr:fix/158-rply-expr branch Mar 13, 2014
@krlmlr krlmlr deleted the krlmlr:fix/158-rply-expr branch Mar 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment