Skip to content

add incomparables arg to merge.data.table and warn about passing arguments to ...#5233

Merged
mattdowle merged 4 commits intomasterfrom
merge_incomparables
Nov 30, 2021
Merged

add incomparables arg to merge.data.table and warn about passing arguments to ...#5233
mattdowle merged 4 commits intomasterfrom
merge_incomparables

Conversation

@ben-schwen
Copy link
Copy Markdown
Member

@ben-schwen ben-schwen commented Oct 23, 2021

Closes #2587
Closes #2593

Implements option 1 and 2 from here

Does not add an additional argument to [.data.table (there are already 17 of them)

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 23, 2021

Codecov Report

Merging #5233 (36d627c) into master (154fcf9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 36d627c differs from pull request most recent head 3b49262. Consider uploading reports for the commit 3b49262 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5233   +/-   ##
=======================================
  Coverage   99.50%   99.50%           
=======================================
  Files          77       77           
  Lines       14631    14643   +12     
=======================================
+ Hits        14559    14571   +12     
  Misses         72       72           
Impacted Files Coverage Δ
R/merge.R 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 154fcf9...3b49262. Read the comment docs.

@mattdowle mattdowle added this to the 1.14.3 milestone Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

inconsistencies between merge.data.frame and merge.data.table merge.data.table ignores incomparables without warnings.

3 participants