Lokal_Profil added a comment.

Is there a particular reason/need to have lojban.org in the tests? If there isn't then simply removing it could make sense.

Also since I'm on mobile and can't leave this content in gerrit:

  • You have referenced the wrong parameter in the docstring for the decorator.
  • Many of the new docstrings don't include epydoc fields for their parameters.

Other than that and the 100 retries for lojban.org the patch looks good and feels like a clear improvement over the old structure.


TASK DETAIL
https://phabricator.wikimedia.org/T151368

EMAIL PREFERENCES
https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Magul, Lokal_Profil
Cc: Lokal_Profil, gerritbot, Dalba, pywikibot-bugs-list, Aklapper, Magul, Th3d3v1ls, Ramalepe, Liugev6, Tbscho, MayS, Lewizho99, Mdupont, JJMC89, Maathavan, jayvdb, Masti, Alchimista, Rxy
_______________________________________________
pywikibot-bugs mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikibot-bugs

Reply via email to