-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEATURE] add in-memory provider to SDK #165
Comments
fwiw @toddbaert I'm looking at this, limited bandwidth, but interested in completing this |
Hey @CommCody ! I've assigned you. It's no problem if this takes some time. I'll keep it assigned to you for now. |
@CommCody are you working on this? |
Mildly, I do have some work on my fork. My time is limited however, so I'm not very actively working on it. |
Can I continue from where you left? |
@NeerajGartia21 if @CommCody has a partial implementation, maybe you'd be better off working on the Java reconnect tests, or on the Error enhancements in JS you recently commented on. |
Cool, that sounds good. |
Hi @toddbaert I was updating my code to further implement this issue, but now I can't build the main branch anymore. the build fails with:
If I disable the following line in
I have the following sdk installed: |
Thanks @CommCody, there was a similar report over the weekend, so looking into this. |
Fwiw I'm running these commands on windows 11. Directly against the dotnet cli. |
@CommCody Hey! Unfortunately this has become a bit of a blocker. If you aren't able to finish this in the next few days, would you mind if I take it over? I'm glad to pick up where you left off if you give me a branch, and will maintain as much of your work as possible, and make you a commit co-author. |
Hi @toddbaert , totally fair My current state of work is here https://github.com/CommCody/dotnet-sdk/tree/add-in-memory-provider From memory:
KR |
Add
InMemoryProvider
to SDK. See spec.Tasks:
InMemoryProvider
.see Java equivalent.
Parent issue: #126
The text was updated successfully, but these errors were encountered: