Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.48%. Comparing base (82f559f) to head (73dc3cc).
Report is 1 commits behind head on master.

❗ Current head 73dc3cc differs from pull request most recent head 7eb8082. Consider uploading reports for the commit 7eb8082 to get more accurate results

@@           Coverage Diff           @@
##           master    #5960   +/-   ##
=======================================
  Coverage   97.48%   97.48%           
=======================================
  Files          80       80           
  Lines       14859    14859           
=======================================
  Hits        14486    14486           
  Misses        373      373           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Thanks! Code change looks correct.

To complete this PR, please:

• Add a NEWS item, citing yourself
• Add a regression test at the end if inst/tests/tests.Rraw. Something along the lines of the MRE you provided in the issue would suffice.

For details: https://github.com/Rdatatable/data.table/blob/master/.github/CONTRIBUTING.md

@MichaelChirico Done! My editor trimmed off some trailing whitespace in a few unrelated lines, hopefully that's not a problem.

Ty @dshemetov, some minor tweaks and we are good to go!

Strange that commit dshemetov@b974ca9 is not being reflected in the PR here...

@MichaelChirico I just noticed that we forgot to add @dshemetov to the Description as contributor.

Can you add @dshemetov as a member to the organization? (following our code of conduct)