Ben Oukhanov
Ben Oukhanov
/sig code-quality
@enp0s3 @AlonaKaplan @EdDev Thank you. I'll take care of it.
I broke by mistake this PR, I wanted to rebase and forgot to do `git rebase --continue` before `git commit --amend --no-edit`. Fixing it now.
@enp0s3 @EdDev @xpivarc Thanks! Can you please review it again?
I see merge conflicts here [1]: ``` Auto-merging tests/utils.go Auto-merging tests/vm_test.go CONFLICT (content): Merge conflict in tests/vm_test.go Automatic merge failed; fix conflicts and then commit the result. ``` I'll resolve...
I rebased it and during rebase there was no merge conflicts. Let's see if it happens again.
@EdDev Thanks a lot :) I'll take care of your review comments.
Sorry, I don't have capacity to work on this PR. Thanks for all reviewers. If someone else wants to take care of it, please do.
Hi Assaf, I think in general we're trying to get rid of `tests/utils.go`. We don't want to add new functions there, but in your case you're just changing it. Do...