search

valkey-io / valkeymodule-rs

Rust valkey SDK for modules
BSD 3-Clause "New" or "Revised" License
35 stars 10 forks source link

Should we use Rust aliases to for compatibility bwn valkey-module and redis-module #10

Open dmitrypol opened 7 months ago

dmitrypol commented 7 months ago

As part of creating new valkey-module crate we forked redis-module and renamed a bunch of things. For example RedisString became ValkeyString. Should we implement Rust aliases https://doc.rust-lang.org/reference/items/type-aliases.html to hep with code compatibility?

If someone built a module in Rust for Redis using redis-module they can update their Cargo.toml to reference valkey-module. But they would have to make numerous updates in code. Aliases will simplify that.

mkmkme commented 4 months ago

I think having those type aliases would be great. If you want, I can come up with a PR