Add support for Dev Containers about nredisstack HOT 10 CLOSED

@haysch We can use what you already have as a base and make changes to fit exactly our needs.
What are you saying?

from nredisstack.

Will tidy it a bit up, and then open a PR.

By the way, thanks for an awesome and active library - your work is highly appreciated 👍

from nredisstack.

@haysch Thanks for the feedback!
Dev Containers sounds like a good idea
Do you have experience with it?

from nredisstack.

I have been using it for a while on a smaller scale, so nothing fancy.

I already have a devcontainer setup for NRedisStack that I have been using for a while as it was somewhat quick and painless to support multiple Redis, and .NET, versions 💡

from nredisstack.

How do we want to support multiple Redis versions?

The pipeline tests three different versions, but the test suite uses a single REDIS env variable or localhost:6379, if env variable is not set.

How do you usually do it today?

In my environment I have spinner up the three versions used by the pipeline, and then specify different test tasks that points to the respective Redis instance.

from nredisstack.

Locally I test against redis-stack:edge. And the CI also checks the other versions for me.
Sounds like what you offer is better.

from nredisstack.

@haysch How's it going? Do you want to send what you have in the meantime and we will continue together?

from nredisstack.

Will have some time in the evening.
Life has been going on 🙂

from nredisstack.

@shacharPash Added an initial commit, but it seems like some tests are failing for v6.2.6 and v7.0.6 - any idea why?

Please let me know what you think, or make any changes as you see fit!

from nredisstack.

completed in #169

from nredisstack.