-
Notifications
You must be signed in to change notification settings - Fork 183
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
Lint seq_len(length(x))
#2577
Comments
I agree! would you like to have a shot at writing a PR? |
Thanks for the offer! Maybe in the future. I have too many things at the moment, and I don't know the internals of lintr... But for reference, are there resources you would recommend to learn how to do it? |
|
that's the right starting point! beyond that the existing linters (and seq_linter() itself) are a great place to start. |
Should be linted to
seq_along(x)
Should not lint:
seq_len(length(x) - 1)
The text was updated successfully, but these errors were encountered: