Xqt created this task. Xqt added projects: Pywikibot, Pywikibot-Scripts, Pywikibot-RfCs. Restricted Application added subscribers: pywikibot-bugs-list, Aklapper.
TASK DESCRIPTION Status ------ create_isbn_edition.py originally written by Geert Van Pamel was added 3 years ago with Pywikibot 7.7 but I am unsure whether this is ever used from this repository and always behind the initial source at https://github.com/geertivp <https://github.com/geertivp/Pywikibot/blob/main/create_isbn_edition.py>. Overall it is hardly maintainable ind keep in sync with it. - the last change <https://github.com/geertivp/Pywikibot/commit/9401f8778c341b56f9bc47bd2acb1cd1034a9ded> wasn't ported to pywikibot - different repositories with different further development are difficult to keep in sync if they after changes - the script uses a lot of global variables which is generally considered bad practice and should be avoided if possible. - the script is in critical risc categoty with a cyclomatic complexity of ~80 - in result the script is vulnerable to unintended changes, difficult to maintain, has reduced modularity and probably performance issues - finally there are a lot of open tasks according this script for further development Proposal -------- I propose to remove `create_isbn_edition.py` from Pywikibot scripts package. It is still available under the repository https://github.com/geertivp/Pywikibot which hold several scripts. With code entry point for foreign scripts <https://doc.wikimedia.org/pywikibot/master/entrypoint.html> introduced in Pywikibot 9.4 the whole script package could be published by the maintainer as a separate package based on Pywikibot like we do it with pywikibot-scripts <https://pypi.org/project/pywikibot-scripts/>. See also T377066 <https://phabricator.wikimedia.org/T377066>. TASK DETAIL https://phabricator.wikimedia.org/T398140 EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/
_______________________________________________ pywikibot-bugs mailing list -- [email protected] To unsubscribe send an email to [email protected]
