-
Notifications
You must be signed in to change notification settings - Fork 160
fix: use 0 for unset auto-renew period #12577
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
Conversation
Signed-off-by: Michael Tinker <[email protected]>
Node: HAPI Test (Restart) Results2 tests 2 ✅ 5m 30s ⏱️ Results for commit db41313. |
Node: HAPI Test (Node Death Reconnect) Results1 tests 1 ✅ 27s ⏱️ For more details on these parsing errors, see this check. Results for commit db41313. |
Node: HAPI Test (Token) Results207 tests 207 ✅ 19m 7s ⏱️ Results for commit db41313. |
Node: HAPI Test (Misc) Results433 tests 423 ✅ 41m 44s ⏱️ Results for commit db41313. |
Node: HAPI Test (Crypto) Results311 tests 311 ✅ 45m 56s ⏱️ Results for commit db41313. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 53m 55s ⏱️ Results for commit db41313. |
Node: Unit Test Results 2 273 files ±0 2 273 suites ±0 2h 41m 44s ⏱️ - 1h 11m 56s Results for commit db41313. ± Comparison against base commit 62d0dc6. This pull request removes 3975 and adds 3745 tests. Note that renamed tests count towards both.
|
Node: HAPI Test (Smart Contract) Results550 tests 549 ✅ 1h 12m 7s ⏱️ For more details on these failures, see this check. Results for commit db41313. |
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:
0
for missing auto-renew period ingetTokenInfo()
result #12576