mirror of
https://github.com/swift-server/RediStack.git
synced 2026-05-03 07:32:28 +00:00
Nathan Harris
c4ff3fc300
Motivation: As originally pointed out in #48, the `map` prefix alone is not enough context into what the method actually does. Since it fails the future, and isn't a mapping function, the name should reflect this. But, this method as-is currently provides little value outside of the client context, so it should not be `public`. If users provide an adequate use case for having it outside of the package, then it can be made public again. Modifications: - Rename: ELF where Value == RESPValue extension `map` to `tryConverting` - Change: `tryConverting` from public to internal Result: A "problematic" method should no longer be available for API users to hurt themselves with.