-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rename VersionedStore#redo to VersionedStore#rollforward #13
Comments
I've thought about the naming divergence between the methods before, but never really put enough thought into it to come up with a better name scheme for it. 😄 I actually chose I'm glad you're trying out the gem, though! I find it a lot of fun to use and the immutable models force me to think about problems differently, rather than rushing in in my usual fashion. I usually use it to manage data structures in server-side persistent processes and client-side with Clearwater. |
Given "redo" is a control statement, would it make sense to rename
VersionedStore#redo
toVersionedStore#rollforward
? That way it's similar to "rollback". Or perhapsVersionedStore#rewind
andVersionedStore#replay
?Just an idea I'm tossing out there. I discovered grand_central recently and gave it a whirl and that caught me by surprise.
The text was updated successfully, but these errors were encountered: