Skip to content

Release v0.10.5#198

Merged
khoroshevskyi merged 3 commits intomasterfrom
dev
Jun 3, 2025
Merged

Release v0.10.5#198
khoroshevskyi merged 3 commits intomasterfrom
dev

Conversation

@khoroshevskyi
Copy link
Member

Changes:

  • Fixed usage stats

TODO:

  • Version of pepdbagent updated in __version__.py file
  • Changelog updated

@khoroshevskyi khoroshevskyi requested a review from Copilot June 3, 2025 18:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR prepares the 0.10.5 release by updating version metadata, improving usage statistical uploads, and upgrading ECS deployment actions in CI.

  • Switched print to _LOGGER.info in upload_usage, added error handling, and reset usage metadata.
  • Modified the count_requests decorator to call the wrapped function once and return its result.
  • Bumped version to 0.10.5 and updated ECS deploy actions from v1 to v2 in workflows.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
bedhost/main.py Replaced stdout print with logger, wrapped usage upload in try/except
bedhost/helpers.py Refactored decorator to store and return the function result
bedhost/_version.py Updated package version to 0.10.5
.github/workflows/deploy-api.yml Upgraded amazon-ecs-deploy-task-definition action to v2
.github/workflows/deploy-api-dev.yml Upgraded amazon-ecs-deploy-task-definition action to v2
Comments suppressed due to low confidence (2)

bedhost/helpers.py:98

  • [nitpick] The name function_result is generic. Consider renaming it to result or response for clarity.
function_result = await func(*args, **kwargs)

bedhost/helpers.py:98

  • Calling the wrapped function before computing usage metrics means errors there will skip metric tracking. If metrics must run regardless of success, move this call into a try/finally or call func after the metric logic.
function_result = await func(*args, **kwargs)

@khoroshevskyi khoroshevskyi merged commit 071b26a into master Jun 3, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants