-
Notifications
You must be signed in to change notification settings - Fork 678
Issue 423: Array items getting lost #219
Conversation
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.
Just need to fix the linter issue in solidity_array_structs.js
@ccowell - in the future please refer to issues and PRs using the org/project#XX syntax - github automatically links them. From the reviewer perspective #423 is a lot easier than checking the 3 ganache repos at random. Also since your PR description didn't restate the issue I had to look at the commit messages to determine that you weren't referring to #423 |
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.
See summary of requested changes from #218 - those all apply here as well. Will rereview after those issues are addressed, but I suspect things will be 👍 at that point.
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.
LGTM!
Will reopen when |
Unable to confirm reported bug.