Skip to content
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

Will there be a migration documentation for v8 ? #446

Open
lorenh opened this issue Jan 7, 2022 · 6 comments
Open

Will there be a migration documentation for v8 ? #446

lorenh opened this issue Jan 7, 2022 · 6 comments

Comments

@lorenh
Copy link

lorenh commented Jan 7, 2022

I see some major changes in the newly released version version 8.0.0 including a renamed IRedisCacheClient to something else. Will you be publishing any upgrade documentation on the new way to do things?

@rlightner
Copy link

Agreed, huge breaking change by removing that interface with zero acknowledgment of it.

@Sveer
Copy link

Sveer commented Jan 10, 2022

+1. Breaking change at 43c2401#diff-70746b527c10a102a9b8ac4d661ef5ef5e3ae5783bcdc60909850da8df4cd229".
"where T : class" - broke working with value types again.

@dotsad
Copy link

dotsad commented Jan 12, 2022

Can some one explain why they added where T : class everywhere? And yes, breaking changes without a slight documentation.

@imperugo
Copy link
Owner

imperugo commented Jan 14, 2022

Hi yes there are some breaking changes, it's a major so breaking changes are welcome, I'm working also on documentation
@dotsad @rlightner this is an open source project, if you like to contribute (also with documentation) you are welcome. Open an PR and I'll be happy to review a merge it.

@imperugo
Copy link
Owner

for your happines, the constraint has been removed, if you download the 8.0.3 you can store a value type directly into redis without any wrapping class.

@dotsad
Copy link

dotsad commented Jan 14, 2022

for your happines, the constraint has been removed, if you download the 8.0.3 you can store a value type directly into redis without any wrapping class.

Wow, thanks. BTW thank you for this library!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants