Skip to content

Fix NatsHybridCacheExtensions TODO#17

Merged
caleblloyd merged 1 commit intohybrid-cache-extensionsfrom
codex/implement-todo-in-natshybridcacheextensions-cs
May 17, 2025
Merged

Fix NatsHybridCacheExtensions TODO#17
caleblloyd merged 1 commit intohybrid-cache-extensionsfrom
codex/implement-todo-in-natshybridcacheextensions-cs

Conversation

@caleblloyd
Copy link
Copy Markdown
Contributor

Summary

  • implement AddNatsHybridCache by registering the serializer factory based on the configured INatsConnection
  • document AddNatsHybridCache and AddNatsHybridCacheSerializerFactory

Testing

  • dotnet test --no-build (fails: dotnet not found)

@caleblloyd caleblloyd merged commit cb74576 into hybrid-cache-extensions May 17, 2025
@caleblloyd caleblloyd deleted the codex/implement-todo-in-natshybridcacheextensions-cs branch May 17, 2025 05:41
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