Add a mask for future %||%, use it

[MichaelChirico]

Uncovered by the new `coalesce_linter():

r-lib/lintr#2767

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.59%. Comparing base (05e3bdf) to head (6116e58).
Report is 8 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6826   +/-   ##
=======================================
  Coverage   98.59%   98.59%           
=======================================
  Files          79       79           
  Lines       14661    14661           
=======================================
  Hits        14455    14455           
  Misses        206      206           

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

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

Generated via commit 6116e58

Download link for the artifact containing the test results: ↓ atime-results.zip

Task	Duration
R setup and installing dependencies	4 minutes and 39 seconds
Installing different package versions	8 minutes and 13 seconds
Running and plotting the test cases	2 minutes and 20 seconds

[rikivillalba]

there is a single instance of get0(... ifnotfound =...) if it is worth

R/data.table.R:1487: if (verbose && !get0('done_units_report', parent.frame(), ifnotfound = FALSE)) {

[MichaelChirico]

Are you thinking to replace get0() with %||%, something like get("done_units_report", parent.frame()) %||% FALSE? I don't have an immediate preference either way... presumably get0() is a bit optimized...

[tdhock]

cool, I did not know about %||% but that is a nice trick