#205: Fix Pylint errors
------------------------+---------------------------------------------------
Reporter: bthiell | Owner: bthiell
Type: defect | Status: assigned
Priority: major | Milestone:
Component: *general* | Version:
Resolution: | Keywords:
------------------------+---------------------------------------------------
Comment (by simko):
WRT 6), both pylint 0.21.1 (on Debian) and 0.18.0 (on SLC5) can
determine the score of `bibindex_engine_stemmer_greek.py` fine for me.
It would be good to understand why it fails for you?
By "masking deeper errors" I meant that people might be led to rely
upon final detected average scores, not paying much attention to side
warnings. Making the final detected score very bad helps to avoid
such potential tendencies.
WRT 7), I see. It would have been better to add new failing tests as
//new// separate unit tests, so that the old tests keep working fine,
and the new tests would serve us in the TDD manner as a reminder that
something has to be done. Kind of like we did for `(U(1) OR SL(2,Z))`
in the regression test suite.
--
Ticket URL: <http://invenio-software.org/ticket/205#comment:13>
Invenio <http://invenio-software.org>
