FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

only assign copied value in := if not null by ben-schwen · Pull Request #5289 · Rdatatable/data.table · GitHub

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

Merged
mattdowle merged 5 commits into
masterfrom
assign_list_NULL
Mar 15, 2022
Merged

mattdowle merged 5 commits into
masterfrom
assign_list_NULL

Conversation

Copy link
Copy Markdown
Member

Closes #5284

codecov Bot commented Dec 12, 2021
edited
Loading

Copy link
Copy Markdown

Codecov Report

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

@@           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 added this to the 1.14.3 milestone Mar 15, 2022
mattdowle merged commit 1fb423b into master Mar 15, 2022
mattdowle deleted the assign_list_NULL branch March 15, 2022 13:00
jangorecki modified the milestones: 1.14.9, 1.15.0 Oct 29, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL