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

--- Comment #187 from Kyle M Hall <k...@bywatersolutions.com> ---
(In reply to Chris Cormack from comment #179)
> Hopefully this new super strict QA standard will be across the board and I
> won't see people pushing patches they wrote skipping qa steps anymore.

I'd like to address this. I'm not in favor of super strict QA standards. What
I'm in favor of are transparent, standardized, and consistently applied QA
rules.

My experience as a developer has informed my work as a QA team member. I have
had to rewrite C4 modules as Koha modules, I've asked others to rewrite C4
modules as Koha modules. Code quality guidelines should be completely divorced
from the necessity or popularity of a bugfix or feature.

I think our coding guidelines need a bit of a revamp to make sure all of the
things we've taken for granted as being necessary to pass QA are actually in
the coding guidelines. If a QA team member fails QA on a patch, the QA'er
should be able to link directly to the reason and explanation for it failing
QA. If the QA'er believes the patch should fail QA but the reason is not part
of the coding guidelines, then a meeting should be called, a vote should be
made, and a new rule added to the guidelines.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to