-
Notifications
You must be signed in to change notification settings - Fork 975
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
merge sort arg document better #4427
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4427 +/- ##
==========================================
+ Coverage 99.47% 99.60% +0.13%
==========================================
Files 75 72 -3
Lines 14808 13917 -891
==========================================
- Hits 14730 13862 -868
+ Misses 78 55 -23
Continue to review full report at Codecov.
|
… NAs in key columns, and in this context of missings in merge, that sentence was more likely to mislead towards putting missings from all.y/all.x up front
…ed as 'added at the end'
…3.5.0. And the nabble link doesn't seem to be working either, so removing the paragraph solves that too.
…only mentioned choosing merge columns
…eems it was fixed in R 3.5.0. And the nabble link doesn't seem to be working either, so removing the paragraph solves that too.
explain difference of sorting NAs in data.table merge
closes #2574
closes #2594