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
Some can't be fixed without breaking backward compatibility, but there should be a clean way to disable them (I didn't investigate, and I had never used Codacy before).
Cc: @senabIsShort, @QuentinSanchez, @MartinDlry, this may be an interesting topic to work on: reading and fixing warnings is a nice way to learn the subtleties of a language.
The text was updated successfully, but these errors were encountered:
moy
added
the
easy?
Probably easy to implement, or WIP almost complete
label
Feb 28, 2019
When I visit that link, it says there are no issues. There are 3 "use of exec" ignored issues. Perhaps they updated their software and fixed some false positives?
By the way, I have found ScrutinizerCI to be a lot more useful than Codacy in other projects.
To a Python novice, this may seem like a reasonable way to default to an empty array for the extras param. What really happens is that this "default" array gets created as a persistent object, and every invocation of my_method that doesn't specify an extras param will be using that same list object—any changes to it will persist and be carried to every other invocation!
See Ned Batchelder's blog post of 14 June 2008 about PyLint, which highlights this problem and suggests some better alternatives.
I've activated codacy for MechanicalSoup.
There seem to be some relevant warnings here:
https://app.codacy.com/app/moy/MechanicalSoup/issues/index
(at least the
value={...}
)Some can't be fixed without breaking backward compatibility, but there should be a clean way to disable them (I didn't investigate, and I had never used Codacy before).
Cc: @senabIsShort, @QuentinSanchez, @MartinDlry, this may be an interesting topic to work on: reading and fixing warnings is a nice way to learn the subtleties of a language.
The text was updated successfully, but these errors were encountered: