-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implement consensus/consensus.go
#24
Comments
assign please |
@DarkLord017 what are the updates on the progress so far? |
Completed half of the functions rest are pending (structs are done)... I'll complete it by today EOD |
@star-gazer111 |
@VeerChaurasia will create a new PR modifying networks.go by tomorrow. You can use it then. |
@DarkLord017 refer #35 |
Okay |
@star-gazer111 sir done hai almost consensus_core me utils file nhi thi uske functions consensus.go me likhu ya consensus core me utils.go ek file bnayi hai uski PR alag se kr du ? |
@star-gazer111 sir consensus.go me getpayload function define nahi hai lekin Helios me he aur uska use aa rha he to vo consensus me likhna he na ? |
cool works |
|
Okay sir |
@DarkLord017 also, before defining anything in a new types.go please do check if we have already defined it somewhere in common or config module so we don't rewrite same thing again. |
okay sir |
@DarkLord017 do you need ChainID function and Baseconfig function(defined in networks.go) outside the package of config ? |
Resolved here #51 . |
No description provided.
The text was updated successfully, but these errors were encountered: