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

Implemented_execution_proof_go_execution_errors_go_#30 #34

Conversation

DarkLord017
Copy link

No description provided.

@DarkLord017
Copy link
Author

@star-gazer111 @gerceboss

@star-gazer111
Copy link
Member

@DarkLord017 please rebase.

@DarkLord017
Copy link
Author

Ok will do

@gerceboss
Copy link
Contributor

@DarkLord017, why does the lint fails?

@DarkLord017
Copy link
Author

@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

return true
}

func getRestPath(p []byte, s int) string {
Copy link
Contributor

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

Copy link
Author

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

@gerceboss
Copy link
Contributor

@DarkLord017 re-request for review whenever you push changes

@DarkLord017
Copy link
Author

@DarkLord017 re-request for review whenever you push changes

Okay sir I will push in few minutes n tag u

@DarkLord017 DarkLord017 force-pushed the Implemented_execution_proof_go_execution_errors_go_#30 branch from 507b692 to 394539b Compare September 27, 2024 20:40
@DarkLord017
Copy link
Author

Made the changes
(The linterwill not pass as i inherited account from types .go in execution which Veer has made a PR for) that's why it is failing
@gerceboss u can review it now

@gerceboss gerceboss merged commit a24a843 into BlocSoc-iitr:dev Sep 28, 2024
1 of 4 checks passed
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.

3 participants