Skip to content

only assign copied value in := if not null#5289

Merged
mattdowle merged 5 commits intomasterfrom
assign_list_NULL
Mar 15, 2022
Merged

only assign copied value in := if not null#5289
mattdowle merged 5 commits intomasterfrom
assign_list_NULL

Conversation

@ben-schwen
Copy link
Copy Markdown
Member

Closes #5284

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 12, 2021

Codecov Report

Merging #5289 (47d54e0) into master (c2e60fe) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5289   +/-   ##
=======================================
  Coverage   99.50%   99.50%           
=======================================
  Files          77       77           
  Lines       14662    14662           
=======================================
  Hits        14590    14590           
  Misses         72       72           
Impacted Files Coverage Δ
R/data.table.R 99.89% <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 c2e60fe...47d54e0. Read the comment docs.

Comment thread inst/tests/tests.Rraw Outdated
@mattdowle mattdowle added this to the 1.14.3 milestone Mar 15, 2022
@mattdowle mattdowle merged commit 1fb423b into master Mar 15, 2022
@mattdowle mattdowle deleted the assign_list_NULL branch March 15, 2022 13:00
@jangorecki jangorecki modified the milestones: 1.14.9, 1.15.0 Oct 29, 2023
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.

When using {} in RHS of :=, sometimes NULL does not remove a column.

4 participants