Page MenuHomePhabricator

Add phan to LiquidThreads extension
Closed, ResolvedPublic

Description

As wmf deployed extension the LiquidThreads extension should run phan on CI to found simple bugs in an earlier state of development.
LiquidThreads extension is not using the common /includes/ or /src/ folder, which makes the config for phan a bit harder.
It also mixed up \Page and \Article at some places which makes it harder to follow the code for phan

Event Timeline

Change 528937 had a related patch set uploaded (by Umherirrender; owner: Umherirrender):
[integration/config@master] [LiquidThreads] Add phan dependency

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

Change 528939 had a related patch set uploaded (by Umherirrender; owner: Umherirrender):
[mediawiki/extensions/LiquidThreads@master] build: Add mediawiki/mediawiki-phan-config

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

Change 528937 merged by jenkins-bot:
[integration/config@master] [LiquidThreads] Add phan dependency

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

Change 528939 merged by jenkins-bot:
[mediawiki/extensions/LiquidThreads@master] build: Add mediawiki/mediawiki-phan-config

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

Umherirrender claimed this task.
Umherirrender triaged this task as Medium priority.