Skip to content
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

Protobuf Changes for Eth2 APIs EventSource Support #8999

Merged
merged 5 commits into from
Jun 7, 2021

Conversation

rauljordan
Copy link
Contributor

What does this PR do? Why is it needed?

Which issues(s) does this PR fix?

Part of the initiative to align with the eth2 standard API, we need to support the events streaming endpoints explained here. We need to support the following schema:

https://ethereum.github.io/eth2.0-APIs/#/Events/eventstream

This PR only contains protobuf changes.

@rauljordan rauljordan requested a review from a team as a code owner June 7, 2021 17:15
@rauljordan rauljordan added the Ready For Review A pull request ready for code review label Jun 7, 2021
@rauljordan rauljordan self-assigned this Jun 7, 2021
@codecov
Copy link

codecov bot commented Jun 7, 2021

Codecov Report

Merging #8999 (2f0f7cd) into develop (26c4e7b) will increase coverage by 0.39%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #8999      +/-   ##
===========================================
+ Coverage    60.56%   60.96%   +0.39%     
===========================================
  Files          531      531              
  Lines        37959    37536     -423     
===========================================
- Hits         22990    22883     -107     
+ Misses       11694    11367     -327     
- Partials      3275     3286      +11     

@prylabs-bulldozer prylabs-bulldozer bot merged commit a81c4ed into develop Jun 7, 2021
@delete-merged-branch delete-merged-branch bot deleted the proto-event-source branch June 7, 2021 19:29
//
// This service is defined in the upstream eth2.0-apis repository (eth2.0-APIs/apis/debug).
service Events {
// StreamEvents --
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe outline this comment more whenever possible?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants