Fixes None from get_version() (#179)

Handle None from get_pkg_version
Adding confirmation to upload_static_assets.py

ARXIVNG-3008

Author: bdc34

Pipfile

@@ -29,6 +29,7 @@ sphinxcontrib-websupport = "*"
 sphinx-autodoc-typehints = "==1.8.0"
 astroid = "<2.2"
 mypy-extensions = "==0.4.2"
+click = "*"
 
 [pipenv]
 allow_prereleases = true

Pipfile.lock

@@ -44,10 +44,11 @@ def get_version(dist_name: str) -> Optional[str]:
         return dist_version
     except ModuleNotFoundError:
         pass
-    try:
-        return get_pkg_version(dist_name)
-    except Exception:
-        pass
+
+    pkv=get_pkg_version(dist_name)
+    if pkv is not None:
+        return pkv
+
     try:
         return get_git_version()
     except ValueError:
@@ -97,7 +98,7 @@ def get_pkg_version(pkg: Any) -> Optional[str]:
     """
     try:
         return pkg_resources.get_distribution(pkg).version
-    except Exception:
+    except:
         return None
 
 

arxiv/release/dist_version.py

@@ -1,8 +1,13 @@
 """Use this to upload static content to S3."""
 
+import click
 import flask_s3
 from arxiv.base.factory import create_web_app
 
 app = create_web_app()
 with app.app_context():
-    flask_s3.create_all(app)
+    sup = app.static_url_path
+    if click.confirm(f'Upload static assets for {sup} to public S3 bucket at {app.config.get("FLASKS3_BUCKET_NAME")}?', default=True):
+        flask_s3.create_all(app)
+    else:
+        print("Aborted.")