Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

"str(i), " might not be needed. (or {} might be needed) #4

Closed
tdrk1980 opened this issue Feb 23, 2019 · 3 comments
Closed

"str(i), " might not be needed. (or {} might be needed) #4

tdrk1980 opened this issue Feb 23, 2019 · 3 comments
Labels

Comments

@tdrk1980
Copy link
Contributor

Hi, thank you for sharing an useful script.
During reading your code, I have found a small issue.
pylint reports "Too many arguments for format string" as below:
image

So, I think "str(i), " might not be needed for this line(L.122).

@cballenar
Copy link
Owner

Thanks for pointing it out. Yes, that str(i) was out of place. It looks like I copied and pasted from a previous line (for shame!). Feel free to submit a pull request next time.

@cballenar cballenar added the bug label Feb 23, 2019
@cballenar
Copy link
Owner

Also, be sure to checkout SlideGrubber. Can be used as a python package and is more complete. (unless you just need to run as a command line, then you're good).

@tdrk1980
Copy link
Contributor Author

Feel free to submit a pull request next time.

OK, I will try it next time. Thank you!

P.S.
This script (with tiny changes) works well on Windows10 / python3.6.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants