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.

Sign up for GitHub

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

Jump to bottom

Ignore trailing whitespace inside strings (optionally?) #1045

Closed
MichaelChirico opened this issue Apr 4, 2022 · 0 comments · Fixed by #1119
Closed

Ignore trailing whitespace inside strings (optionally?) #1045

MichaelChirico opened this issue Apr 4, 2022 · 0 comments · Fixed by #1119
Labels
feature a feature request or enhancement
Milestone
3.0.0

Comments

@MichaelChirico
Copy link
Collaborator

Currently this lints:

lintr::lint("a <- '\n  \n'", lintr::trailing_whitespace_linter())
# <text>:2:1: style: Trailing whitespace is superfluous.
#   
# ^~

But there could be any number of reasons to include trailing whitespace inside a string.
@AshesITR AshesITR mentioned this issue May 2, 2022
Merged
@AshesITR AshesITR added this to the 3.0.0 milestone May 2, 2022
@AshesITR AshesITR added the feature a feature request or enhancement label May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature a feature request or enhancement
Projects
None yet
Milestone
3.0.0
Development

Successfully merging a pull request may close this issue.

Ignore trailing whitespace in strings by default r-lib/lintr
2 participants
@AshesITR @MichaelChirico