You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on May 5, 2019. It is now read-only.
If anything I would do it the other way around: keep merge!, since that's the familiar name for this operation, and deprecate hcat!, since that's more reminiscent of Julia matrix terminology.
Sounds good to me! It's also easier to change the internal function names than it is to change all the code out in the wild that uses merge!. When I open the PR for this I'll add something to the docs to help users choose between hcat, merge!, and full outer join which all allow users to merge tables but do so in different ways. I just found out about merge! while reviewing tests for the #30 refactor.
AFAICT
merge!
andhcat!
do the same thing. Should we exporthcat!
and deprecatemerge!
?The text was updated successfully, but these errors were encountered: