-
Notifications
You must be signed in to change notification settings - Fork 273
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
Comment spacing rule #761
Comment spacing rule #761
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.
Looks great! Could you add it to the README? @chavacava for a final review.
Hi @qascade, thanks for the PR |
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.
I've made some minor changes, added some test cases, and left some comments
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.
I've made some modifications and documented the rule.
Thanks @qascade ! |
This PR will close issue #735.
Did you add tests?
Yes, tests were added
Does your code follow the coding style of the rest of the repository?
Yes, the code was written while keeping the coding style of the repo in mind. But, if you find any decrepancy, please feel free to request a change in the code review.
Does the Travis build passes?
Yes, the build is passing along with all the tests.