pylint: Reenable R1723 no-else-break

* R1723: Unnecessary "elif" after "break", remove the leading "el" from "elif" (no-else-break)

fdroidserver/common.py

@@ -1803,7 +1803,7 @@ def parse_androidmanifests(paths, app):
                                 if re.match(r'.*[\'"\s]{flavour}[\'"\s].*\{{.*'.format(flavour=flavour), line):
                                     inside_required_flavour = 2
                                     break
-                                elif re.match(r'.*[\'"\s]{flavour}[\'"\s].*'.format(flavour=flavour), line):
+                                if re.match(r'.*[\'"\s]{flavour}[\'"\s].*'.format(flavour=flavour), line):
                                     inside_required_flavour = 1
                                     break
 

fdroidserver/deploy.py

@@ -623,7 +623,7 @@ def upload_apk_to_virustotal(virustotal_apikey, packageName, apkName, hash,
                                 + str(response['positives']) + ' times:'
                                 + '\n\t' + response['permalink'])
             break
-        elif r.status_code == 204:
+        if r.status_code == 204:
             logging.warning(_('virustotal.com is rate limiting, waiting to retry...'))
             time.sleep(30)  # wait for public API rate limiting
 

pyproject.toml

@@ -67,7 +67,6 @@ disable = [
   "missing-class-docstring",
   "missing-function-docstring",
   "missing-module-docstring",
-  "no-else-break",
   "no-else-continue",
   "no-else-raise",
   "no-else-return",