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.

Sign up for GitHub

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

Jump to bottom

Rename sync_committee_aggregate > sync_aggregate #2796

Merged
merged 1 commit into from
Jan 13, 2022
Merged

Rename sync_committee_aggregate > sync_aggregate #2796

merged 1 commit into from
Jan 13, 2022

Conversation

etan-status
Copy link
Contributor

This renames the sync_committee_aggregate field of LightClientUpdate
to sync_aggregate for consistency with the terminology in the rest of
the spec.

@etan-status
Rename sync_committee_aggregate > sync_aggregate
48e19b1 
This renames the `sync_committee_aggregate` field of `LightClientUpdate`
to `sync_aggregate` for consistency with the terminology in the rest of
the spec.
ralexstokes
ralexstokes approved these changes Jan 11, 2022
View reviewed changes
Copy link
Member

@ralexstokes ralexstokes left a comment

Choose a reason for hiding this comment

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

lgtm

@hwwhww hwwhww added general:presentation Presentation (as opposed to content) scope:light-clients labels Jan 13, 2022
@hwwhww hwwhww merged commit 16c350e into ethereum:dev Jan 13, 2022
@etan-status etan-status deleted the lc-aggregate branch January 13, 2022 14:57
@tersec tersec mentioned this pull request Feb 1, 2022
Merged
@g11tech g11tech mentioned this pull request Feb 3, 2022
Closed
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hwwhww hwwhww hwwhww approved these changes

@ralexstokes ralexstokes ralexstokes approved these changes

Assignees
No one assigned
Labels
general:presentation Presentation (as opposed to content) scope:light-clients
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@etan-status @ralexstokes @hwwhww