jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/350314 )
Change subject: Use raise ImportError instead of sys.exit(1) when requests is not found
......................................................................
Use raise ImportError instead of sys.exit(1) when requests is not found
Bug: T76238
Change-Id: Iab6aa68c0022825e8ce0127f76d2df191a25917a
---
M pwb.py
1 file changed, 2 insertions(+), 3 deletions(-)
Approvals:
jenkins-bot: Verified
Xqt: Looks good to me, approved
diff --git a/pwb.py b/pwb.py
index ca7179f..14b4c09 100755
--- a/pwb.py
+++ b/pwb.py
@@ -161,9 +161,8 @@
requests = None
if not requests:
- print("Python module requests is required.")
- print("Try running 'pip install requests'.")
- sys.exit(1)
+ raise ImportError("Python module 'requests' is required.\n"
+ "Try running 'pip install requests'.")
del requests
--
To view, visit
https://gerrit.wikimedia.org/r/350314
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iab6aa68c0022825e8ce0127f76d2df191a25917a
Gerrit-PatchSet: 3
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Framawiki <framawiki(a)tools.wmflabs.org>
Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: Magul <tomasz.magulski(a)gmail.com>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>