Topic on Talk:Continuous integration/Codehealth Pipeline

KHarlan (WMF) (talkcontribs)

As you might have noticed, we've deployed an updated job that now calculates PHPUnit code coverage based solely on unit tests (tests which extend MediaWikiUnitTestCase and are in `tests/phpunit/unit`). We've been doing this for MediaWiki core for a while and just rolled out the same rules for extensions for consistency and also to speed up the feedback loop. task T230423 has some more information.

DCausse (WMF) (talkcontribs)

our current phpcs rule impose that all test PHP files have some @covers tags, should this now be restricted to tests/phpunit/unit files only?

KHarlan (WMF) (talkcontribs)

@DCausse (WMF) that's a good point; perhaps eventually, but the codehealth pipeline and SonarQube integration is still experimental, so I think we could leave it as is for now. It would probably be useful to have a more formal discussion on this subject.