Skip to content

Conversation

@TimTaylor
Copy link
Contributor

@TimTaylor TimTaylor commented Oct 29, 2025

closes #2916

supersedes #2919. Main difference from before I borked that PR is the renaming of the argument from allow_print to allow_paren_print as suggested in https://github.com/r-lib/lintr/pull/2919/files#r2350158023.

@TimTaylor TimTaylor changed the title feat: argument to disable linting of ( for auto-printing feat: argument to disable linting of ( for auto-printing Oct 29, 2025
@TimTaylor TimTaylor marked this pull request as ready for review October 29, 2025 11:24
@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.24%. Comparing base (65229d8) to head (61424b7).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2962   +/-   ##
=======================================
  Coverage   99.24%   99.24%           
=======================================
  Files         129      129           
  Lines        7279     7282    +3     
=======================================
+ Hits         7224     7227    +3     
  Misses         55       55           

☔ 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.

Copy link
Collaborator

@MichaelChirico MichaelChirico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM after one more round of improving the parameter description. Thanks!

@MichaelChirico MichaelChirico merged commit a6ac90b into r-lib:main Oct 29, 2025
21 checks passed
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.

FR: add argument to disable linting of ( for auto-printing in implicit_assignment_linter

2 participants