Skip to content

Commit

Permalink
op-bindings: delete ast package (ethereum-optimism#10299)
Browse files Browse the repository at this point in the history
This package canonicalized the ast ids that were placed
in the bindings `more` files because any change in the
smart contracts would cause a massive diff in the storage
layout for all storage layouts as they used the ast ids
to represent types. We needed to canonicalize them to have
deterministic bindings generation, otherwise CI would generate
the bindings in a non deterministic way. As we adopted new
solidity features, it would find edge cases in our ast canonicalize
algo, breaking CI. This code helped us for a long time but
now it is no longer used and can be retired.
  • Loading branch information
tynes authored Apr 25, 2024
1 parent 35f2989 commit c5a2ec4
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 1,177 deletions.
143 changes: 0 additions & 143 deletions op-bindings/ast/canonicalize.go

This file was deleted.

54 changes: 0 additions & 54 deletions op-bindings/ast/canonicalize_test.go

This file was deleted.

Loading

0 comments on commit c5a2ec4

Please sign in to comment.