From 8503e57916f203b96ac58755986ba5961275c5f9 Mon Sep 17 00:00:00 2001 From: Simon McVittie Date: Sat, 1 Dec 2018 13:37:38 +0000 Subject: po: Add failing test for Debian bug #911356 Signed-off-by: Simon McVittie --- ...content_instead_of_skipping_if_previously_translated.mdwn | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) (limited to 'doc/todo') diff --git a/doc/todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn b/doc/todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn index 57386ad75..749551ccc 100644 --- a/doc/todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn +++ b/doc/todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn @@ -95,5 +95,13 @@ > I've filed this in Debian GNU/Linux at https://bugs.debian.org/911356 — [[lamby]], Thu, 18 Oct 2018 20:18:58 -0400 - - +>> As I said on the Debian bug, I think we badly need test coverage for +>> this sort of thing, otherwise it *will* regress. The po plugin is +>> relatively complicated and hooks into lots of places in ikiwiki, +>> and I don't think any of the active ikiwiki maintainers use it +>> themselves, which means it can easily break (or have pre-existing +>> bugs) without anyone realising. +>> +>> For now I've added a failing test-case for this particular bug. +>> I'll try to review this soon and understand the po plugin, the patch, +>> why the bug exists and why the patch fixes it. --[[smcv]] -- cgit v1.2.3