Skip to content

Conversation

@spalladino
Copy link
Contributor

  • Remove legacy buildBlock() method from FullNodeCheckpointsBuilder
  • Migrate EpochPruneWatcher to use checkpoint-based APIs (startCheckpoint() + buildBlock())
  • Add ICheckpointsBuilder and ICheckpointBlockBuilder interfaces to stdlib to avoid circular dependency between slasher and validator-client
  • Delete unused IFullNodeBlockBuilder interface and BuildBlockResult type

@spalladino spalladino added the ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure label Jan 16, 2026
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/63ecc108bef73a4c�63ecc108bef73a4c8;;�): yarn-project/end-to-end/scripts/run_test.sh compose src/guides/writing_an_account_contract.test.ts (166s) (code: 1) (\033Santiago Palladino\033: refactor: remove block builder in favor of checkpoint builder)

@spalladino spalladino force-pushed the palla/remove-l2-block branch from 64b3a14 to 50bc3b1 Compare January 18, 2026 02:14
@spalladino spalladino force-pushed the palla/remove-block-builder branch from 6d438c8 to 361b937 Compare January 18, 2026 02:15
@spalladino spalladino merged commit 1a14d33 into palla/remove-l2-block Jan 18, 2026
6 of 7 checks passed
@spalladino spalladino deleted the palla/remove-block-builder branch January 18, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants