Home Original page

Implement RedisValue.Length for all underlying storage kinds by mgravell · Pull Request #2370 · StackExchange/StackExchange.Redis

@mgravell

fix #2368

  • implement Length() for other encodings (using format layout)
  • unify format code
  • switch to C# 11 for u8 strings (needed a few "scoped" modifiers adding)
  • tests for format and Length

@mgravell 

- implement Length() for other encodings (using format layout)
- unify format code
- switch to C# 11 for u8 strings (needed a few "scoped" modifiers adding)
- tests for format and Length

@mgravell

NickCraver

@mgravell

@NickCraver

If we want to go .NET 7, will need to bump Ubuntu too it looks like, not sure what's on their images (have to run in a few but can investigate this afternoon and tidy to get unblocked if you're on other stuff)

@mgravell

@NickCraver

NickCraver

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good 👍