-
Notifications
You must be signed in to change notification settings - Fork 37
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
chore: Conformance To Style Standards With RuboCop & TravisCI #71
chore: Conformance To Style Standards With RuboCop & TravisCI #71
Conversation
e19c200
to
7c36ca9
Compare
Hello @alanunruh, |
Hello @alanunruh, |
Hey @thinkingserious! Thanks for letting me help and resolve some issues! I was reviewing the requirements for 5 points needed for the swag and it looks like one of the options is to contribute two (2) ‘difficulty: medium’ issues. I completed this PR and sendgrid/sendgrid-nodejs#774 and the leaderboard is only saying I have 3 points. No biggie, but just want to point that out if you have some auto calculations doing the leaderboard. Thanks again for letting me contribute to SendGrid and thanks for all you do with the community! Happy to help in anyway I can. |
Hello @alanunruh, |
7c36ca9
to
c643727
Compare
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!
Fixes #68
Checklist
Short description of what this PR does:
rubocop.yml
so that developers may runrubocop
in the CLI.