Page MenuHomePhabricator

Migrate WikimediaTools to the new validator framework
Closed, ResolvedPublic

Description

Migrate the existing checker / suggester code to instead use the new validator framework. We need to add,

  • Replace Crosswatch checkers with RegexInsertable validators
  • Add others as needed

We'll need to update the YAML configuration file located here - https://phabricator.wikimedia.org/diffusion/GTWN/browse/master/groups/

Documentation: https://www.mediawiki.org/wiki/Help:Extension:Translate/Validators
Example patch: https://gerrit.wikimedia.org/r/c/translatewiki/+/549930

Event Timeline

Change 571484 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Update Wikimedia tools to use the new validator framework

https://gerrit.wikimedia.org/r/571484

abi_ triaged this task as Medium priority.Feb 11 2020, 1:46 PM
abi_ moved this task from Backlog to Configuration changes on the translatewiki.net board.

Change 571484 merged by jenkins-bot:
[translatewiki@master] Update Wikimedia tools to use the new validator framework

https://gerrit.wikimedia.org/r/571484

Nikerabbit added a subscriber: Kizule.

@Zoranzoki21 Please don't close tasks which are on our sprint board (Language-Team (Language-2021-April-June)). We have a process, including QA, which the tasks have to go through before being closed.

Change 575821 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[mediawiki/extensions/Translate@master] Add NumericalParameterValidator

https://gerrit.wikimedia.org/r/575821

Change 575822 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Replace NumericalParameterInsertable with a validator

https://gerrit.wikimedia.org/r/575822

Change 575823 had a related patch set uploaded (by Abijeet Patro; owner: Abijeet Patro):
[translatewiki@master] Replace NumericalParameter insertable with NumericalParameter validator

https://gerrit.wikimedia.org/r/575823

Change 575822 abandoned by Abijeet Patro:
Replace NumericalParameterInsertable with a validator

Reason:
See Ibbde5a0dc4bdf30a2071d2d3c8357eec0de61bb3

https://gerrit.wikimedia.org/r/575822

Change 575821 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Add NumericalParameterValidator

https://gerrit.wikimedia.org/r/575821

Change 575823 merged by jenkins-bot:
[translatewiki@master] Replace NumericalParameter insertable with NumericalParameter validator

https://gerrit.wikimedia.org/r/575823

This has been deployed on the server. Resolving the task.