-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
refactor(Message): combine patch methods #6071
Conversation
This needs a rebase. |
bc4f39f
to
e7ae4f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
,i feel like if statements make more sense than assigning the property to itself, idk if it really matters though
Ended up doing it for everything, makes the code overall much easier to read. |
As a wise, young man once said, this needs a rebase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what's major about this, but well 😅
9d6fdeb
to
b2fca46
Compare
Please describe the changes this PR makes and why it should be merged:
Two patch methods, both private, second often gets missed during changes. No more!
Status and versioning classification: