Skip to content

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

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