-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add lintr for automated PR reviews #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. How do you feel about a linter for max 80 characters per line? I tend to try and stick with it (if not consistently in this package) but perhaps it's not needed.
Looking over the code, the another good linter might be for snake_case variable names. |
Yes, feel free to re-enable If you want to fix the current code and re-enable these linters, I would recommend you start by re-enabling the relevant linter, then run Also note that |
No description provided.