refactor(action): rename function for cache-control

moneymeets · Nov 8, 2023 · da2d30f · da2d30f
1 parent 1b1eacf
commit da2d30f
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/s3_artifact/action.py b/s3_artifact/action.py
@@ -58,7 +58,7 @@ def deploy(config: S3ArtifactConfig, artifacts_s3_path: str, environment: str) -
 
 
 def upload(config: S3ArtifactConfig, target: str) -> Sequence[str]:
-    def _prepare_metadata_command(cache_config: S3ArtifactCustomMetadataConfig):
+    def _prepare_cache_control_and_content_type_command(cache_config: S3ArtifactCustomMetadataConfig):
         content_type = cache_config.mime_type or DEFAULT_MIME_TYPES[Path(cache_config.path).suffix]
         content_type_option = f"--content-type '{content_type}'"
 
@@ -72,7 +72,10 @@ def _get_default_cache_control():
 
     return (
         f"aws s3 sync {config.local_artifacts_path} {target} {_get_default_cache_control()} {S3_SYNC_OPTIONS}",
-        *(_prepare_metadata_command(custom_metadata) for custom_metadata in config.custom_metadata),
+        *(
+            _prepare_cache_control_and_content_type_command(custom_metadata)
+            for custom_metadata in config.custom_metadata
+        ),
     )