[Koha-bugs] [Bug 10539] Remove unused unit tests in lib/KohaTest

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10539

Galen Charlton <gmcharlt at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gmcharlt at gmail.com

--- Comment #13 from Galen Charlton <gmcharlt at gmail.com> ---
(In reply to Srdjan Jankovic from comment #7)
> A philosophical question - changes like this, consisting of only removing
> unused/unwanted code, can those be self-signed, and go to QA directly?

A philosophical answer -- no, for two reasons.  If a patch removes an unused
something, having at least two pairs of eyes can help verify that (a) the
something was in fact unused and (b) the something was /completely/ removed. 
Second, it gives folks time to jump in case they question the removal.

-- 
You are receiving this mail because:
You are watching all bug changes.