-
Notifications
You must be signed in to change notification settings - Fork 81
fix: allowed callTracer to return empty array if no call actions found #3768
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: allowed callTracer to return empty array if no call actions found #3768
Conversation
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) |
Test Results 22 files + 1 271 suites - 2 36m 52s ⏱️ + 3m 24s For more details on these failures, see this check. Results for commit 09ae045. ± Comparison against base commit fca86cc. ♻️ This comment has been updated with latest results. |
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
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
ecea0d7
to
09ae045
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.
Approved, but left a comment.
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
Description:
This PR allows callTracer to return an empty array if no call actions are found, effectively preventing the deconstruction of an undefined object.
Related issue(s):
Fixes #3751
Notes for reviewer:
Checklist