handle terminal switch() in return_linter()

[MichaelChirico]

Closes #2343.

cc @MEO265

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (acbfd77) 98.54% compared to head (9b6bfaa) 98.55%.

❗ Current head 9b6bfaa differs from pull request most recent head e2ecb04. Consider uploading reports for the commit e2ecb04 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2422   +/-   ##
=======================================
  Coverage   98.54%   98.55%           
=======================================
  Files         126      126           
  Lines        5721     5735   +14     
=======================================
+ Hits         5638     5652   +14     
  Misses         83       83           

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

[MichaelChirico]

😆 Definitely my most unexpected time getting hit with the commented_code_linter() 🔨

switch(x, ...) | expr[1]: switch; expr[2]: x

Pretty impressive that parses as valid code!

[AshesITR]

😆 Definitely my most unexpected time getting hit with the commented_code_linter() 🔨

switch(x, ...) | expr[1]: switch; expr[2]: x

Pretty impressive that parses as valid code!

I like to prepend the code by #>, which generally kills parsing due to a missing LHS of > ;)

[MichaelChirico]

😆 Definitely my most unexpected time getting hit with the commented_code_linter() 🔨

switch(x, ...) | expr[1]: switch; expr[2]: x

Pretty impressive that parses as valid code!

I like to prepend the code by #>, which generally kills parsing due to a missing LHS of > ;)

right, but I would reserve that for code that's actually intended as code. In this case, I was writing a comment to explain the XPath

[AshesITR]

Awesome!

[MichaelChirico]

roxygen test handy already 🤭