do nothing for now

y-scope · Dec 12, 2024 · ce5a667 · ce5a667
1 parent f41c558
commit ce5a667
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/components/clp-py-utils/clp_py_utils/s3_utils.py b/components/clp-py-utils/clp_py_utils/s3_utils.py
@@ -9,10 +9,7 @@
 
 
 def verify_s3_config_for_archive_output(s3_config: S3Config) -> Result:
-    # TODO: need to verify:
-    # 1. Have write priveldge so archive can be compressed
-    # 2. Have read priviledge so archive can be readed
-    # 3. bucket and region are the same, this should run into issue but not sure
+    # TODO: properly verify the s3 config
     return Result(success=True)