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

Created utils.go #41

Conversation

DarkLord017
Copy link

Please check all the functions

@DarkLord017
Copy link
Author

Implemented consensus/utils.go & corrected consensus/rpc & consensus/consensus_core.go

Please check the functions in utils.go
@star-gazer111

@DarkLord017 DarkLord017 marked this pull request as ready for review September 20, 2024 02:30
@DarkLord017 DarkLord017 marked this pull request as draft September 20, 2024 07:30
@DarkLord017
Copy link
Author

DarkLord017 commented Sep 20, 2024

@star-gazer111 sir consensus_core me kuch chize implemented nahi hai to usme changes krdiye hai aur public bhi krdiya hai
np sir
PR abhi vapis draft krdi hai .....
krdiya update sir

@DarkLord017 DarkLord017 marked this pull request as ready for review September 20, 2024 08:11
Comment on lines 16 to 28
type Forks struct {
Genesis ForkVersion
Altair ForkVersion
Bellatrix ForkVersion
Capella ForkVersion
Deneb ForkVersion
}

type ForkVersion struct {
Epoch uint64
ForkVersion [4]byte
}

Copy link
Member

Choose a reason for hiding this comment

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

this stuff is already implemented here. Why are you reimplementing the same stuff?

Copy link
Author

Choose a reason for hiding this comment

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

Ok sir I'll correct it

@DarkLord017 DarkLord017 marked this pull request as draft September 21, 2024 03:28
@DarkLord017 DarkLord017 marked this pull request as ready for review September 21, 2024 07:54
@DarkLord017 DarkLord017 marked this pull request as draft September 21, 2024 07:55
@DarkLord017
Copy link
Author

Please refer to PR #51

@star-gazer111
Copy link
Member

@DarkLord017 this PR is for?

@DarkLord017
Copy link
Author

It is already merged (it was for consensus/utils.go) refer #51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants