Merge pull request #44 from DataCrunch-io/fix/remove-print

fix: removed print and minor changes

Author: tamirse

.github/workflows/publish_package.yml

@@ -5,7 +5,7 @@ name: Upload Python Package
 
 on:
   release:
-    types: [published, created]
+    types: [published]
 
 jobs:
   deploy:

CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+
+- Trigger publish package github action only when a released is published
+- Async inference example: longer `sleep()` duration when polling for inference status
+
+### Fixed
+
+- Removed a forgotten and redundant `print`
+
 ## [1.13.0] - 2025-05-21
 
 ### Changed

datacrunch/containers/containers.py

@@ -921,7 +921,6 @@ def get_secrets(self) -> List[Secret]:
             List[Secret]: List of all secrets.
         """
         response = self.client.get(SECRETS_ENDPOINT)
-        print(response.json())
         return [Secret.from_dict(secret) for secret in response.json()]
 
     def create_secret(self, name: str, value: str) -> None:

examples/containers/calling_the_endpoint_with_inference_key_async.py

@@ -30,7 +30,7 @@
 # Poll for status until completion
 while async_inference_execution.status() != AsyncStatus.Completed:
     print(async_inference_execution.status_json())
-    sleep(1)
+    sleep(5)
 
 # Print the response
 print(async_inference_execution.output())