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

added skeleton controller classes and main #55

Merged
merged 1 commit into from
Dec 5, 2018
Merged

added skeleton controller classes and main #55

merged 1 commit into from
Dec 5, 2018

Conversation

jrhea
Copy link
Contributor

@jrhea jrhea commented Dec 5, 2018 •
edited
Loading

resolves #40. added basic skeleton classes and a main.

@jrhea
resolves #40
a329d0a 
added basic skeleton classes and a main
schroedingerscode
schroedingerscode approved these changes Dec 5, 2018
View reviewed changes
Copy link
Contributor

@schroedingerscode schroedingerscode left a comment

Choose a reason for hiding this comment

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

Once we decide on the final form for what our event loop and delegates look like, we can define an abstract class or interface for the relevant functions (i.e. init, start, and stop).

@jrhea jrhea changed the title resolves #40 added skeleton controller classes and main Dec 5, 2018
@jrhea jrhea assigned jrhea and unassigned jrhea Dec 5, 2018
@akhila-raju akhila-raju merged commit efd2ad6 into Consensys:master Dec 5, 2018
zilm13 added a commit to zilm13/teku that referenced this pull request May 16, 2024
@zilm13
Switch from extra subnet count in discovery to total subnet count lik…
9fd3f7b 
…e others do (Consensys#55)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akhila-raju akhila-raju akhila-raju approved these changes

@schroedingerscode schroedingerscode schroedingerscode approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create Basic Event Loop
3 participants
@jrhea @akhila-raju @schroedingerscode