From 168ef1854a3cd9b06edbd4a5597c73793f18f07f Mon Sep 17 00:00:00 2001 From: AntiCompositeNumber Date: Tue, 21 Jul 2020 16:34:33 -0400 Subject: [PATCH] Url._url_check should return None There's no point in returning True if it's never checked and won't ever be False. Implicitly returning None or raising an exception is more idiomatic. --- waybackpy/wrapper.py | 1 - 1 file changed, 1 deletion(-) diff --git a/waybackpy/wrapper.py b/waybackpy/wrapper.py index a5eef01..6c47b58 100644 --- a/waybackpy/wrapper.py +++ b/waybackpy/wrapper.py @@ -64,7 +64,6 @@ class Url: """Check for common URL problems.""" if "." not in self.url: raise URLError("'%s' is not a vaild URL." % self.url) - return True def _clean_url(self): """Fix the URL, if possible."""