]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
comment
authorSimon McVittie <smcv@debian.org>
Sun, 3 Feb 2019 18:53:23 +0000 (18:53 +0000)
committerSimon McVittie <smcv@debian.org>
Sun, 3 Feb 2019 18:53:23 +0000 (18:53 +0000)
doc/todo/_Add_ikiwiki_PO-Plugin_to_ignore_languages.mdwn

index e3a57e1845889629af333e5ad25a18b21fb23cae..690f89d5dbd0583092e12424c4b4930d3effe373 100644 (file)
@@ -7,3 +7,16 @@ normally, however, HTML creation will be disabled and no reference to
 these translations will be visible on the website.
 
 See also downstream bug report in tails: <https://labs.riseup.net/code/issues/15355>
+
+> From the downstream bug report it appears this branch is not yet
+> considered ready (as of February 2019), so I haven't attempted to
+> review it.
+>
+> As with everything else in the `po` plugin: I would really want to see
+> some regression test coverage before merging this, because as far as I'm
+> aware none of the ikiwiki maintainers use the `po` plugin, so we are
+> not able to detect regressions other than via the automated tests. That
+> isn't a good situation to be in for a plugin that uses `IkiWiki::inject`
+> to monkey-patch ikiwiki internals. If this plugin is important to you,
+> more test coverage would be very much appreciated.
+> --[[smcv]]