jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
cleanup: remove unused 'error' parameter from History.log() method

Bug: T380693
Change-Id: Iade518b3d24c8bdf470b891f1bc8c9bc569e4099
---
M scripts/weblinkchecker.py
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/weblinkchecker.py b/scripts/weblinkchecker.py
index 14174b8..daaa880 100755
--- a/scripts/weblinkchecker.py
+++ b/scripts/weblinkchecker.py
@@ -363,7 +363,7 @@
# no saved history exists yet, or history dump broken
self.history_dict = {}

- def log(self, url, error, containing_page, archive_url) -> None:
+ def log(self, url, containing_page, archive_url) -> None:
"""Log an error report to a text file in the deadlinks subdirectory."""
if archive_url:
error_report = f'* {url} ([{archive_url} archive])\n'
@@ -411,7 +411,7 @@
pywikibot.warning(
f'get_closest_memento_url({url}) failed: {e}')
archive_url = None
- self.log(url, error, page, archive_url)
+ self.log(url, page, archive_url)
else:
self.history_dict[url] = [(page.title(), now, error)]


To view, visit change 1097448. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Iade518b3d24c8bdf470b891f1bc8c9bc569e4099
Gerrit-Change-Number: 1097448
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info@gno.de>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot