Skip to content
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

PEP-8 + String formatting updates #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vkmrishad
Copy link

@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

SendGridDX commented Oct 20, 2018

CLA assistant check
All committers have signed the CLA.

@vkmrishad
Copy link
Author

@misterdorm,

@vkmrishad
Copy link
Author

@misterdorm
Please add available labels and merge this PR

@@ -33,5 +33,5 @@ def kill(self):
if str(e).find("ResourceNotFoundException") is not -1:
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

Choose a reason for hiding this comment

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

#14

@vkmrishad
Copy link
Author

@42B
Thank you

@42B
Copy link

42B commented Oct 26, 2018

@vkmrishad

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

@vkmrishad
Copy link
Author

vkmrishad commented Oct 26, 2018

@42B
Closes #14

@vkmrishad
Copy link
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