Page MenuHomePhabricator

Inject Config to ORESService, convert tests to unit tests
Open, Needs TriagePublic

Description

Remove some usages of globals in ORES service, in favor of injecting Config object.

Event Timeline

kostajh created this task.Sep 10 2019, 8:40 AM
Restricted Application added a project: Machine-Learning-Team. · View Herald TranscriptSep 10 2019, 8:40 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 510166 had a related patch set uploaded (by Kosta Harlan; owner: Kosta Harlan):
[mediawiki/extensions/ORES@master] Inject Config to ORESService, convert test to unit test

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

ACraze moved this task from Lift Wing to ORES on the Machine-Learning-Team board.Jan 19 2021, 10:26 PM

Change 510166 abandoned by Kosta Harlan:
[mediawiki/extensions/ORES@master] Inject Config to ORESService, convert test to unit test

Reason:

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