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

Anchor pre-PR: Modularize validator store #6771

Merged
merged 25 commits into from
Jan 9, 2025
Merged

Anchor pre-PR: Modularize validator store #6771

merged 25 commits into from
Jan 9, 2025

Conversation

dknopik
Copy link
Member

@dknopik dknopik commented Jan 8, 2025 •
edited
Loading

Please merge #6768 and #6772 first

jxs and others added 21 commits December 16, 2024 16:04
@jxs
pass slots_per_epoch at runtime
0ecd6c3
@jxs
remove generic E from unrequired types
6f3fe95
@jxs
Merge branch 'unstable' of github.com:sigp/lighthouse into remove-e-f…
a9cc2fb 
…rom-validator-client
@dknopik
move validator_store to lighthouse_validator_store
30998cd
@dknopik
make validator_store into a trait
f539f85
@dknopik
further reduce dependencies
e473f63
@dknopik
remove environment dependency on beacon_node_fallback
45ce473
@dknopik @ThreeHrSleep
Manually pull in some changes from tracing-integration (thanks sayan!)
be95852 
Co-authored-by: ThreeHrSleep <[email protected]>
@dknopik
remove environment from validator_services
77eca5a
@dknopik
unify boost factor accessors
9b7c160
@dknopik
add builder for DutiesService
b49845a
@dknopik @ThreeHrSleep
Manually merge tracing PR for beacon_node_fallback
79367de 
Co-authored-by: ThreeHrSleep <[email protected]>
@dknopik
Fix chain_spec for BlockService
838eed6
@jxs
address review
6533a67
@jxs
remove PhantomData from SyncDutiesMap
36a4d2f
@dknopik
Merge branch 'remove-e-from-validator-client' into modularize-validat…
c5ab120 
…or-store
@dknopik
fix tests
ebfe234
@dknopik
correct test
07dbf15
@dknopik
Add E to ValidatorStore as associated type
39aec1b
@dknopik
fix tests
e0da923
@dknopik
derive Clone for ValidatorStore's Error and required sub-errors
4510b73
@mergify Mergify
Copy link

mergify bot commented Jan 8, 2025

⚠️ The sha of the head commit of this PR conflicts with #6705. Mergify cannot evaluate rules on this PR. ⚠️

@mergify Mergify
Copy link

mergify bot commented Jan 8, 2025

⚠️ The sha of the head commit of this PR conflicts with #6705. Mergify cannot evaluate rules on this PR. ⚠️

@jimmygchen jimmygchen added the anchor Anchor client label Jan 8, 2025
@mergify mergify bot mentioned this pull request Jan 9, 2025
Draft
@jimmygchen jimmygchen merged commit 64eb848 into sigp:anchor Jan 9, 2025
36 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
anchor Anchor client
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dknopik @jimmygchen @jxs