Skip to content

Conversation

cheatfate
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jun 2, 2023

Unit Test Results

         9 files  ±0    1 071 suites  ±0   36m 16s ⏱️ +48s
  3 699 tests ±0    3 420 ✔️ ±0  279 💤 ±0  0 ±0 
15 763 runs  ±0  15 458 ✔️ ±0  305 💤 ±0  0 ±0 

Results for commit 24ec004. ± Comparison against base commit a3229a6.

♻️ This comment has been updated with latest results.

@@ -522,3 +525,53 @@ proc installKeymanagerHandlers*(router: var RestRouter, host: KeymanagerHost) =
response.data.add handleRemoveValidatorReq(host, key)

return RestApiResponse.jsonResponsePlain(response)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link to upstream API?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specification changes got merged into master with ethereum/keymanager-APIs@ee334a4 but there no link yet.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specification changes got merged into master with ethereum/keymanager-APIs@ee334a4 but there no link yet.

Also, ethereum/keymanager-APIs#58 or that is a link, and better than nothing.

@cheatfate cheatfate force-pushed the km-voluntary-exit branch from 9807d6b to 24ec004 Compare June 12, 2023 21:49
@arnetheduck arnetheduck merged commit 927180f into unstable Jun 14, 2023
@arnetheduck arnetheduck deleted the km-voluntary-exit branch June 14, 2023 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants