You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A variable format is used which is a reserved keyword in python.
Change that to something like output_format.
Fixed the issue but need to test out, will do a pr soon! 😄
The text was updated successfully, but these errors were encountered:
cg-cnu
changed the title
fix format varible
fix format variable
Oct 13, 2017
I wouldn't be too fussed about this particular use, partly because it's originally one from cmds.playblast(format="") but also because the function referenced by the keyword isn't used in the scope within which it is being used.
In general, if shadowing a keyword improves overall readability, I think it's ok. Put another way, if adhering to PEP08 reduces readability, then I'd say it's safe to ignore the particular warning.
I agree with you, that's why I left it in the initial pr. Since we are planning for pylinting on travis, thought this might error out. I am not sure if we can tune the pylinter to ignore this case 🙂
A variable
format
is used which is a reserved keyword in python.Change that to something like
output_format
.Fixed the issue but need to test out, will do a pr soon! 😄
The text was updated successfully, but these errors were encountered: