Page MenuHomePhabricator

autoload.ide.php needs to be updated or removed
Closed, ResolvedPublic

Description

rMW3ca13774e244: PHPUnit autoload file for PhpStorm

autoload.ide.php has been technical debt ever since it was added, given that it was mostly copy-paste from phpunit.php and doMaintenance.php (as documented in the commit message).

With https://gerrit.wikimedia.org/r/#/c/373174/ this file is now falling behind because it does weird hacky stuff. We should figure out how to get rid of it and support IDEs with the normal phpunit running mechanism.

Event Timeline

Legoktm created this task.Aug 23 2017, 5:31 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 23 2017, 5:31 AM

Is this task considered TD because it's essentially duplicate code (reading into the "mostly copy-paste" comment)?

With https://gerrit.wikimedia.org/r/c/mediawiki/core/+/451636 this file will fall even more behind, so I think we can just remove it now, given that it's seemingly unmaintained.

Is this task considered TD because it's essentially duplicate code (reading into the "mostly copy-paste" comment)?

Yep.

Change 451681 had a related patch set uploaded (by simetrical; owner: simetrical):
[mediawiki/core@master] Delete autoload.ide.php

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

Change 451681 merged by jenkins-bot:
[mediawiki/core@master] Delete autoload.ide.php

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

Legoktm closed this task as Resolved.Aug 12 2018, 8:12 AM
Legoktm assigned this task to Simetrical.