-
Notifications
You must be signed in to change notification settings - Fork 8
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
Implemented_execution_proof_go_execution_errors_go_#30 #34
Implemented_execution_proof_go_execution_errors_go_#30 #34
Conversation
@DarkLord017 please rebase. |
Ok will do |
@DarkLord017, why does the lint fails? |
I made a pr when some consensus files were pending maybe that's the reason I will rebase it n then make a commit |
execution/proof.go
Outdated
return true | ||
} | ||
|
||
func getRestPath(p []byte, s int) string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a comment that this is dead code, just to make it easier for future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I'll do it
@DarkLord017 re-request for review whenever you push changes |
Okay sir I will push in few minutes n tag u |
507b692
to
394539b
Compare
Made the changes |
No description provided.