-
Notifications
You must be signed in to change notification settings - Fork 714
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: Clean up sendgrid.py #844
Conversation
Minor Change
Codecov Report
@@ Coverage Diff @@
## master #844 +/- ##
==========================================
+ Coverage 80.02% 82.41% +2.38%
==========================================
Files 70 70
Lines 2018 2303 +285
Branches 339 338 -1
==========================================
+ Hits 1615 1898 +283
- Misses 204 206 +2
Partials 199 199
Continue to review full report at Codecov.
|
sendgrid/sendgrid.py
Outdated
import python_http_client | ||
|
||
|
||
class SendGridAPIClient(object): |
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.
if you'll use a good linter like python-black. It'll automatically handle all these things and also the two-line gap before python classes are best practices. You may have a look at other files too.
Hello @thinkingserious Thanks for merging my PR and your appreciation. I've filled out the form. Regards |
Minor Change
Fixes #X
Checklist
Short description of what this PR does:
If you have questions, please send an email to Twilio SendGrid, or file a GitHub Issue in this repository.