Skip to content

Removing OGS Package#228

Merged
ben-chain merged 2 commits intomasterfrom
feat/YAS271/remove-ogs
Aug 25, 2020
Merged

Removing OGS Package#228
ben-chain merged 2 commits intomasterfrom
feat/YAS271/remove-ogs

Conversation

@ben-chain
Copy link
Copy Markdown
Collaborator

Description

The OGS package has largely been deprecated and is replicated in another repo, so we can go ahead and remove from the monorepo. This PR does that!

Contributing Agreement

@K-Ho
Copy link
Copy Markdown
Contributor

K-Ho commented Aug 25, 2020

Copy link
Copy Markdown
Contributor

@K-Ho K-Ho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

goodbye OGS!! lgtm

@ben-chain ben-chain merged commit 4bdd490 into master Aug 25, 2020
@gakonst gakonst deleted the feat/YAS271/remove-ogs branch March 18, 2021 15:02
snario pushed a commit that referenced this pull request Apr 14, 2021
* Final gas optimizations for CTC

* delete and re-init

* Fix issue with tests failing

* Use encoding/decoding from core-utils

* dev: Use eth-optimism/core-utils where applicable (#281)

* Use core-utils where applicable

* Further reduce utils

* Update ovmCREATEEOA.spec.ts

Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>

* remove unused import

* Revert "remove unused import"

This reverts commit 1df87926e7a3ca8ebdf7448ad5f0395fe7766ff7.

* actually fix imports

* A few more comments

* address PR feedback - add comments, renaming vars

* Optimization: Conditional Monotonicity Checking (#307)

* trying it

* cleanup

* remove duplicate comment

* fix docstring

* fix var name

* missing period

Co-authored-by: Ben Jones <ben@pseudonym.party>
Co-authored-by: Maurelian <maurelian@protonmail.ch>
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>
Co-authored-by: Kevin Ho <kevinjho1996@gmail.com>
bap2pecs pushed a commit to babylonlabs-io/optimism that referenced this pull request Jul 31, 2024
Inphi pushed a commit that referenced this pull request Aug 16, 2024
Updates DelayedWETH to use call instead of transfer because
transfer only sends along 2300 gas which will cause the transfer
to fail if the owner contract uses lots of gas in the fallback
function. Adds tests that demonstrate that the function now works
appropriately under any reasonable amount of gas usage. In theory
this means the recovery function can be reentered but this isn't
an issue because the function is authenticated and is recovering
all of the ETH in the contract anyway.
samlaf pushed a commit to samlaf/optimism that referenced this pull request Aug 20, 2024
…#228)

Updates DelayedWETH to use call instead of transfer because
transfer only sends along 2300 gas which will cause the transfer
to fail if the owner contract uses lots of gas in the fallback
function. Adds tests that demonstrate that the function now works
appropriately under any reasonable amount of gas usage. In theory
this means the recovery function can be reentered but this isn't
an issue because the function is authenticated and is recovering
all of the ETH in the contract anyway.
theochap pushed a commit that referenced this pull request Dec 10, 2025
* feat(client): Add `current_output_root` to block executor

Adds a function to the block executor that allows for generating the
current output root, based on the parent timestamp and current trie.

* feat(client): Account + Account storage hinting in `TrieDB`

Introcues the `TrieDBHinter` trait, which outlines an interface for
sending hints to the host for trie account proofs and storage slot
proofs. This enables `kona-host` to use `eth_getProof` rather than the
very limited `debug_dbGet` for fetching account proofs during L2
execution.
Zena-park added a commit to tokamak-network/optimism that referenced this pull request Dec 30, 2025
theochap pushed a commit that referenced this pull request Jan 15, 2026
### Description

Adds small docs for `op-alloy-rpc-types`
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.

2 participants