-
Notifications
You must be signed in to change notification settings - Fork 135
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
nit: better distinguish between ConsumerValidatorBytePrefix
and ConsumerValidatorsBytePrefix
#2087
Comments
@insumity, I would love to work on this issue. |
@aditipatelpro Sorry for the late reply. We were focusing on the ICS 6.0.0 release and we missed your comment. We are gladly accepting external contributions. Please let us know if you still want to take this over so that I can assign it to you. Thanks. |
@mpoke, I am still excited and available to work on this issue. thank you. |
@aditipatelpro Thanks. I assigned the issue to you. Please let us know if you have any questions. |
@mpoke Was this issue fixed in the I can't seem to find either |
@aditipatelpro the way keys are stored got changed a bit. Look for |
@mpoke, thank you for clarifying, I will get started with new key names. |
Hi @mpoke, Unfortunately, I don't have the bandwidth to work on this issue currently. |
Problem
Byte prefixes
ConsumerValidatorsBytePrefix
andConsumerValidatorBytePrefix
almost have the same name although they're used when storing vastly different things.This issue is just a reminder to clean up their naming so it's not as confusing.
The text was updated successfully, but these errors were encountered: