Skip to content

Conversation

@LiTe1991
Copy link
Contributor

No description provided.

@LiTe1991
Copy link
Contributor Author

@hangy review?

@LiTe1991 LiTe1991 changed the title request async flush instead of blocking flush feat: request async flush instead of blocking flush Mar 17, 2025
@hangy hangy requested a review from Copilot March 17, 2025 12:43
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 updates the ApplicationInsightsTarget to use an asynchronous flush method instead of a blocking flush.

  • Removes the blocking flush logic and conditional delay
  • Implements an asynchronous flush with a continuation to handle the completion

@hangy hangy changed the base branch from main to dev March 17, 2025 12:45
@hangy hangy merged commit 0e5ef14 into hangy:dev Mar 17, 2025
2 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 17, 2025
# [1.1.0-beta.1](v1.0.0...v1.1.0-beta.1) (2025-03-17)

### Bug Fixes

* **test:** Explicity set `IncludeGdc=true` in test cases that require GDC ([1b4e247](1b4e247))

### Features

* request async flush instead of blocking flush ([#48](#48)) ([0e5ef14](0e5ef14))
@github-actions
Copy link

🎉 This PR is included in version 1.1.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Mar 18, 2025
# [1.1.0](v1.0.0...v1.1.0) (2025-03-18)

### Bug Fixes

* Remove redundant `ContextProperties` property ([2b75da1](2b75da1)), closes [#49](#49)
* **test:** Explicity set `IncludeGdc=true` in test cases that require GDC ([1b4e247](1b4e247))

### Features

* request async flush instead of blocking flush ([#48](#48)) ([0e5ef14](0e5ef14))
@github-actions
Copy link

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants