-
Notifications
You must be signed in to change notification settings - Fork 57
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 OutputNote variants #586
Comments
This will actually be more than just renaming. After this the |
@polydez - after my refactoring in #645, I think the variant names could be: pub enum OutputNote {
Full(Note),
Header(NoteHeader),
} Also, in your PR, it might make sense to make the refactoring I mentioned in #614 (comment) as @Dominik1999 is out today and he probably won't get to this until tomorrow. |
Closed as resolved by #673 |
What should be done?
Rename
Public
toFull
andPrivate
toMinimal
How should it be done?
Straight forward renaming
When is this task done?
When the enum variants are renamed in the whole codebase
Additional context
#572 (comment)
The text was updated successfully, but these errors were encountered: