-
Notifications
You must be signed in to change notification settings - Fork 40
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
fix: Avoid populating order.ordering
with empties.
#618
Conversation
85feaec
to
64c5dc2
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.
That's a silly mistake! Sorry Shahzad, and sorry for not spotting it when we looked this over again last week!
Cheers for fixing
We don't do mistakes, just happy accidents haha. Was a fun learning opportunity to watch out for those for myself aswell. |
Hunch was corrent, as mentioned in the standup it was indeed a `make` where it would populate the array with empties and then again append on top of them.)
suggestion on another PR.
64c5dc2
to
e7eb47a
Compare
- Resolves: sourcenetwork#589 - Description: Hunch mentioned in last week's meeting was correct, it was indeed a `make` where it would populate the array with empties and then append being called on top of it. Also removed the function `IsEmpty()`, we used to check for empty elements previously which was introduced in sourcenetwork#481.
Relevant issue(s)
Resolves #589
Description
Hunch mentioned in last week's meeting was correct, it was indeed a
make
where it would populate the array with empties and then append being called on top of it.Also removed the function
IsEmpty()
, we used to check for empty elements previously which was introduced in #481.Tasks
How has this been tested?
make test
and CI checks.Specify the platform(s) on which this was tested: