Skip to content

Fix missing ConfigureAwait in async call#19

Merged
caleblloyd merged 1 commit intohybrid-cache-extensionsfrom
codex/add-configureawait-to-async-calls
May 18, 2025
Merged

Fix missing ConfigureAwait in async call#19
caleblloyd merged 1 commit intohybrid-cache-extensionsfrom
codex/add-configureawait-to-async-calls

Conversation

@caleblloyd
Copy link
Copy Markdown
Contributor

Summary

  • ensure awaited NATS KV retrieval uses ConfigureAwait(false)

Testing

  • dotnet --version (fails: command not found)

@caleblloyd caleblloyd merged commit fd22783 into hybrid-cache-extensions May 18, 2025
@caleblloyd caleblloyd deleted the codex/add-configureawait-to-async-calls branch May 18, 2025 03:19
caleblloyd added a commit that referenced this pull request May 18, 2025
* hybrid cache extension structure

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>

* Document NATS hybrid cache extensions (#17)

* update

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>

* Add NatsHybridCache extension tests and update project refs (#18)

* rename

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>

* Add ConfigureAwait to async call (#19)

* test: increase extension coverage (#20)

* Use AddNatsHybridCache in test utilities (#21)

* remove dupicative test

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>

* cleanup

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>

---------

Signed-off-by: Caleb Lloyd <caleb@boxbuild.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant