Summary
ConfirmEdit (CAPTCHA extension) CI is blocked by Phan failures on the master branch. This is due to the merge of bf464b94c6d38a894b890a2d25d7e7cff7c3866e
Technical details
- https://integration.wikimedia.org/ci/job/mwext-phan-vendor-php83/246/console shows the Phan failure experienced during gate-and-submit. The relevant excerpt is pasted below:
20:51:43 includes/Hooks/Handlers/MakeGlobalVariablesScriptHookHandler.php:37 PhanRedefinedClassReference Saw reference to \MobileContext declared at ../../extensions/MobileFrontend/includes/MobileContext.php:16 which is also declared at ../../extensions/VisualEditor/.phan/stubs/MobileContext.php:7. This may lead to confusing errors. It may be possible to exclude the class that isn't used with exclude_file_list. In addition to normal ways to suppress issues, this issue type can be suppressed on either of the class definitions if it is impractical to exclude one file. 20:51:43 includes/Hooks/Handlers/MakeGlobalVariablesScriptHookHandler.php:58 PhanRedefinedClassReference Saw reference to \MobileContext declared at ../../extensions/MobileFrontend/includes/MobileContext.php:16 which is also declared at ../../extensions/VisualEditor/.phan/stubs/MobileContext.php:7. This may lead to confusing errors. It may be possible to exclude the class that isn't used with exclude_file_list. In addition to normal ways to suppress issues, this issue type can be suppressed on either of the class definitions if it is impractical to exclude one file.
Acceptance criteria
- ConfirmEdit (CAPTCHA extension) CI is unblocked