-
Notifications
You must be signed in to change notification settings - Fork 117
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: Cherry pick: remove gas cost check (#13031) #13078
Conversation
Signed-off-by: Michael Heinrichs <netopyr@users.noreply.github.com> (cherry picked from commit 1c18387)
Node: HAPI Test (Restart) Results2 tests 2 ✅ 5m 18s ⏱️ Results for commit c45d0e7. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Node Death Reconnect) Results1 tests 1 ✅ 20s ⏱️ For more details on these parsing errors, see this check. Results for commit c45d0e7. |
Node: HAPI Test (Token) Results228 tests 227 ✅ 17m 58s ⏱️ Results for commit c45d0e7. |
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! Thanks @netopyr
Node: HAPI Test (Crypto) Results335 tests 335 ✅ 38m 54s ⏱️ Results for commit c45d0e7. |
Node: HAPI Test (Misc) Results457 tests 447 ✅ 39m 8s ⏱️ For more details on these parsing errors, see this check. Results for commit c45d0e7. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 54m 43s ⏱️ Results for commit c45d0e7. |
Node: HAPI Test (Smart Contract) Results587 tests 587 ✅ 1h 11m 13s ⏱️ Results for commit c45d0e7. ♻️ This comment has been updated with latest results. |
Node: Unit Test Results 2 267 files ±0 2 267 suites ±0 2h 9m 24s ⏱️ - 1h 44m 30s Results for commit c45d0e7. ± Comparison against base commit 161bb5d. This pull request removes 3988 and adds 3752 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Description:
Cherry-pick of #13031
Related issue(s):
Closes #13030