-
Notifications
You must be signed in to change notification settings - Fork 33
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: Resolve linting issues. #64
Conversation
Signed-off-by: Ali Yahya <amyahya@gmail.com>
Signed-off-by: Ali Yahya <amyahya@gmail.com>
Signed-off-by: Ali Yahya <amyahya@gmail.com>
Signed-off-by: Ali Yahya <amyahya@gmail.com>
Signed-off-by: Ali Yahya <amyahya@gmail.com>
Hello @mrala, |
Not a problem, @thinkingserious! Just a heads up, I believe this also resolves #60. |
Awesome, thanks for the heads up @mrala! |
Hello @mrala, |
Hello @mrala, |
Resolves #63.
I have made a material change to the repo (functionality, testing, spelling, grammar)
I have read the [Contribution Guide] and my PR follows them.
I updated my branch with the master branch.
I have added tests that prove my fix is effective or that my feature works
I have added necessary documentation about the functionality in the appropriate .md file
I have added in line documentation to the code I modified
Clean up rubopcop issues
If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.