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
spaces_inside_linter ignores trailing comments #638
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.
LGTM, thanks for catching!
this darn travis thing. going to close this PR & reopen on a dummy branch |
…/lintr into spaces-inside-trailing-comment
@jimhester sorry for the bother... searched around a bit, maybe you could remove travis from branch protection? jazzband/pip-tools#1229 (comment) more instructions here: |
Looks like the build failures actually expose a bugged edge-case: Multiple spaces still cause a lint. |
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.
Need to fix for multiple trailing spaces followed by #
as well.
…/lintr into spaces-inside-trailing-comment
good catch; fixed |
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.
LGTM
Closes #636