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

Feature/merge bedrock #1343

Merged
merged 9,407 commits into from
Aug 14, 2023
Merged

Conversation

guoshijiang
Copy link
Contributor

Goals of PR

Core changes:

  • finish bedrock code development

Notes:

  • no

Related Issues:

  • no

hamdiallam and others added 30 commits June 9, 2023 15:04
…gaslimit

contracts-bedrock: deploy time checks
Cleans up the typescript dockerfile. This should eventually
be refactored more but works for now.
fix(docs): Update contracts directory references
The `actor-tests` package is no longer used and can be deleted.
Removes an old script used for generating replica configs.
This is very out of date and has not been used in awhile.
The latest release as of today, see foundry-rs/forge-std@e8a047e
This ensures to bump the dep for both contracts-bedrock and contracts-periphery.
…itignore

chore(indexer): Update git ignroe to ignore binaries
OptimismBot and others added 21 commits June 16, 2023 16:07
…e.contracts

feat(op-bindings) add standardbridge and crossdomainmessenger to bindings
Pack the creation timestamp in the storage slot along
with the address of the dispute games so that the
information can be accessed easily by services that
query the contract.

A service looking for recent dispute games can
binary search or linear search backwards using the
`gameAtIndex` function. It is guaranteed that the
timestamp will always increase or be equal as the
index increases.
…ease-fix

chore(ops): Fix ci-builder release action
…lete_message_relayer_changeset

fix(monorepo): Delete message relayer changeset
feat(ctb): Move the Style Guide into a new doc
lint

delete dead code

changeset

fix: gas estimate without decimals

lint again

fix: Fix the allowance bug that happens if you don't override the signer

lint again

fix: Use isSigner instead

fix: missing a ?
* feat(ctb): Refactor LibPosition.sol to rep gindex

Generalized Indices are a more succinct and easy to understand model for
positions in the dispute game tree. Implementation is often more
efficient too and can avoid branching logic for tree traversal.

* update .gas-snapshot

* update .gas-snapshot

* Fix `FaultDisputeGame` tests

* lint; gas snapshot

* Update packages/contracts-bedrock/contracts/dispute/lib/LibPosition.sol

Co-authored-by: clabby <[email protected]>

* opt rightIndex

* opt rightIndex

* opt depth

* update gas-snapshot

---------

Co-authored-by: clabby <[email protected]>
ops/scripts: Make restored scripts executable.
@guoshijiang guoshijiang merged commit 0d13741 into combined-mantle-op-bedrock Aug 14, 2023
1 of 2 checks passed
@guoshijiang guoshijiang deleted the feature/merge-bedrock branch August 14, 2023 10:32
@guoshijiang guoshijiang restored the feature/merge-bedrock branch August 14, 2023 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.