r111283 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r111282‎ | r111283 | r111284 >
Date:07:33, 12 February 2012
Author:siebrand
Status:ok (Comments)
Tags:
Comment:
Update command documentation.
Modified paths:
  • /trunk/extensions/Translate/scripts/sync-group.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Translate/scripts/sync-group.php
——@@ -21,7 +21,7 @@
2222 function showUsage() {
2323 STDERR( <<<EOT
2424 Options:
25 - --group Comma separated list of group IDs
 25+ --group Comma separated list of group IDs (cannot use groupprefix)
2626 --groupprefix Prefix of group IDs to be exported message groups (cannot use
2727 group)
2828 --lang Comma separated list of language codes or *

Sign-offs

UserFlagDate
SVGinspected10:10, 13 February 2012

Comments

#Comment by SVG (talk | contribs)   10:12, 13 February 2012

--group should be --groups. Would it be possible to add --skipgroups when using --groupprefix?

#Comment by Siebrand (talk | contribs)   19:41, 13 February 2012

Possibly. Patches are appreciated. I don't see a reason to implement it at the moment.

#Comment by Siebrand (talk | contribs)   19:46, 13 February 2012

Why should it be --groups? Works just fine for me as is, and requires typing of one less character.

#Comment by SVG (talk | contribs)   20:00, 13 February 2012

It allows using more than one group. But okay, if you don't want to change it.

Status & tagging log