New coalesce_linter for encouraging %||%

[MichaelChirico]

Closes #2246. Using coalesce_linter() over null_coalescing_linter() to facilitate future extensions to other types of coalescing.

[AshesITR]

Doesn't coalesce imply NULLs are replaced?

[AshesITR]

We could later expand this to ifelse(), if_else(), fifelse() and case_when(), all of which I've seen abused to coalesce.

Similarly the vectorized variants for dplyr if_else(is.na(x), y, x) == coalesce(x, y) and data.table fifelse(is.na(x), y, x) == fcoalesce(x, y).

[MichaelChirico]

We could later expand this to ifelse(), if_else(), fifelse() and case_when(), all of which I've seen abused to coalesce.

Similarly the vectorized variants for dplyr if_else(is.na(x), y, x) == coalesce(x, y) and data.table fifelse(is.na(x), y, x) == fcoalesce(x, y).

So maybe this should just be called coalesce_linter() after all, to anticipate future use cases, WDYT @IndrajeetPatil?

[codecov]

Codecov Report

Attention: Patch coverage is 97.77778% with 1 line in your changes missing coverage. Please review.

Project coverage is 99.59%. Comparing base (3a1e48b) to head (0495900).

Files with missing lines	Patch %	Lines
R/utils.R	50.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2767   +/-   ##
=======================================
  Coverage   99.59%   99.59%           
=======================================
  Files         125      126    +1     
  Lines        6835     6869   +34     
=======================================
+ Hits         6807     6841   +34     
  Misses         28       28           

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