Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possibly remove <<- from the default valid operators in assignment_linter #2717

Open
MichaelChirico opened this issue Feb 7, 2025 · 0 comments
Labels
blocked-external Issues that are blocked by something outside of lintr's direct control breaking change ☠️ API change likely to affect existing code consistency 🍎 🍏
Milestone

Comments

@MichaelChirico
Copy link
Collaborator

Follow-up to #2711. I think the messaging and signature of assignment_linter() will be cleaner if there's only one valid operator by default.

I also think it's worth noting the conflict between the undesirable_operator_linter() defaults and the assignment_linter() defaults.

Filing this issue for now so as not to introduce one more breaking change in the current release.

Upstream issue to watch:

tidyverse/style#239

@MichaelChirico MichaelChirico added breaking change ☠️ API change likely to affect existing code consistency 🍎 🍏 labels Feb 7, 2025
@MichaelChirico MichaelChirico added this to the 3.3.0 milestone Feb 12, 2025
@MichaelChirico MichaelChirico added the blocked-external Issues that are blocked by something outside of lintr's direct control label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked-external Issues that are blocked by something outside of lintr's direct control breaking change ☠️ API change likely to affect existing code consistency 🍎 🍏
Projects
None yet
Development

No branches or pull requests

1 participant