Dedent package processing

We only need to be in the context manager for the file reading.

Author: QuLogic

python/build.py

@@ -23,70 +23,68 @@
     with path.open('r') as fin:
         package = safe_load(fin)
 
-        # Divide the yml files into sections based on the section tag...
-        if 'section' not in package:
-            package['section'] = 'miscellaneous'
-        package['section'] = section_names[package['section'].lower()]
-
-        print(f"  {package['repo']} -> {package['section']}")
-        try:
-            package['user'], package['repo_name'] = package['repo'].split('/')
-        except:
-            raise Warning('Package.repo is not in correct format', package)
-            continue
-        package['conda_package'] = package.get('conda_package', package['repo_name'])
-        package['pypi_name'] = package.get('pypi_name', package['repo_name'])
-
-        if package.get('badges'):
-            package['badges'] = [x.strip() for x in package['badges'].split(',')]
+    # Divide the yml files into sections based on the section tag...
+    if 'section' not in package:
+        package['section'] = 'miscellaneous'
+    package['section'] = section_names[package['section'].lower()]
+
+    print(f"  {package['repo']} -> {package['section']}")
+    try:
+        package['user'], package['repo_name'] = package['repo'].split('/')
+    except:
+        raise Warning('Package.repo is not in correct format', package)
+        continue
+    package['conda_package'] = package.get('conda_package', package['repo_name'])
+    package['pypi_name'] = package.get('pypi_name', package['repo_name'])
+
+    if package.get('badges'):
+        package['badges'] = [x.strip() for x in package['badges'].split(',')]
+    else:
+        package['badges'] = ['pypi', 'conda']
+
+    needs_newline = False
+    if 'pypi' in package['badges']:
+        needs_newline = True
+        print('    pypi: ', end='', flush=True)
+        response = requests.get(f"https://pypi.org/pypi/{package['pypi_name']}/json/")
+        if response.status_code == 200:
+            print('found')
         else:
-            package['badges'] = ['pypi', 'conda']
-
-        needs_newline = False
-        if 'pypi' in package['badges']:
-            needs_newline = True
-            print('    pypi: ', end='', flush=True)
-            response = requests.get(
-                f"https://pypi.org/pypi/{package['pypi_name']}/json/")
-            if response.status_code == 200:
-                print('found')
-            else:
-                print('not found')
-                package['badges'].remove('pypi')
-        if package.get('conda_channel') and 'conda' not in package['badges']:
-            package['badges'].append('conda')
-        if 'conda_channel' not in package:
-            package['conda_channel'] = 'conda-forge'
-        if 'conda' in package['badges']:
-            needs_newline = True
-            print('    conda: ', end='')
-            response = requests.get(
-                f"https://anaconda.org/{package['conda_channel']}/{package['conda_package']}/",
-                allow_redirects=False)
-            if response.status_code == 200:
-                print('found', end='')
-            else:
-                print('not found', end='')
-                package['badges'].remove('conda')
-        if needs_newline:
-            print()
-
-        if package.get('sponsors') and 'sponsor' not in package['badges']:
-            package['badges'].append('sponsor')
-        if package.get('site') and 'site' not in package['badges'] and 'rtd' not in package['badges']:
-            package['badges'].append('site')
-        if package.get('dormant') and 'dormant' not in package['badges']:
-            package['badges'].append('dormant')
-        
-
-        if 'rtd' in package['badges'] and 'rtd_name' not in package:
-            package['rtd_name'] = package['repo_name']
-
-        if 'site' in package['badges']:
-            if 'site' not in package:
-                package['site'] = f'https://{package["repo_name"]}.org'
-            else:
-                package['site'] = package['site'].rstrip('/')
+            print('not found')
+            package['badges'].remove('pypi')
+    if package.get('conda_channel') and 'conda' not in package['badges']:
+        package['badges'].append('conda')
+    if 'conda_channel' not in package:
+        package['conda_channel'] = 'conda-forge'
+    if 'conda' in package['badges']:
+        needs_newline = True
+        print('    conda: ', end='')
+        response = requests.get(
+            f"https://anaconda.org/{package['conda_channel']}/{package['conda_package']}/",
+            allow_redirects=False)
+        if response.status_code == 200:
+            print('found', end='')
+        else:
+            print('not found', end='')
+            package['badges'].remove('conda')
+    if needs_newline:
+        print()
+
+    if package.get('sponsors') and 'sponsor' not in package['badges']:
+        package['badges'].append('sponsor')
+    if package.get('site') and 'site' not in package['badges'] and 'rtd' not in package['badges']:
+        package['badges'].append('site')
+    if package.get('dormant') and 'dormant' not in package['badges']:
+        package['badges'].append('dormant')
+
+    if 'rtd' in package['badges'] and 'rtd_name' not in package:
+        package['rtd_name'] = package['repo_name']
+
+    if 'site' in package['badges']:
+        if 'site' not in package:
+            package['site'] = f'https://{package["repo_name"]}.org'
+        else:
+            package['site'] = package['site'].rstrip('/')
 
     config[package['section']].append(package)