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

Fix build #7

Conversation

Gabriel-Trintinalia
Copy link
Collaborator

  • Besu references were being resolved to arithmetization. I had to set transitive = false in the dependency to make it work. There is probably a smarter way to resolve this.
  • The sequencer was being built using the main branch and not arith-dev. There were some adjustments in tracer.traceEndConflation() I don’t know if they are correct though.
  • There was an error with SimulationValidator (?) I need to look into this
  • Some references to the sequencer package yet in the code
  • Remove the spillings.toml from the code. Should we have this file on the arithmetization repo?
  • the line-limits file seems to be outdated. I had to include EXP = 32760 to make it work.

Signed-off-by: Gabriel-Trintinalia <[email protected]>
@Gabriel-Trintinalia Gabriel-Trintinalia merged commit 550f28b into Consensys:feat/issue-3/arithmetization-removal Apr 23, 2024
4 of 6 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.

1 participant