Skip to content

PEP-8 + String formatting updates#12

Open
vkmrishad wants to merge 3 commits intosendgrid:masterfrom
vkmrishad:patch-1
Open

PEP-8 + String formatting updates#12
vkmrishad wants to merge 3 commits intosendgrid:masterfrom
vkmrishad:patch-1

Conversation

@vkmrishad
Copy link
Copy Markdown

@vkmrishad vkmrishad commented Oct 20, 2018

Closes #14

Checklist

  • 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 inline documentation to the code I modified

Short description of what this PR does:

  • String formatted and updated
  • PEP-8 updated

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

@SendGridDX
Copy link
Copy Markdown

SendGridDX commented Oct 20, 2018

CLA assistant check
All committers have signed the CLA.

@vkmrishad
Copy link
Copy Markdown
Author

@misterdorm,

@vkmrishad
Copy link
Copy Markdown
Author

@misterdorm
Please add available labels and merge this PR

KLog.log("could not find function '%s', dequeueing task" % self.func)
else:
KLog.log("could not delete function '%s', unknown error: %s" % str(e), "critical")
KLog.log("could not delete function '%s', unknown error: %s" % e, "critical")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#14

@vkmrishad
Copy link
Copy Markdown
Author

@42B
Thank you

@42B
Copy link
Copy Markdown

42B commented Oct 26, 2018

@vkmrishad

You might want to update your PR description to close that issue as well.

@vkmrishad
Copy link
Copy Markdown
Author

vkmrishad commented Oct 26, 2018

@42B
Closes #14

@vkmrishad
Copy link
Copy Markdown
Author

@thinkingserious
Can you please check this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TypeError: not enough arguments for format string

4 participants