Skip to content

Adding rollback tagging step in github actions for uat and testnet#281

Merged
willmeister merged 4 commits intomasterfrom
tag_rollback_images
Sep 24, 2020
Merged

Adding rollback tagging step in github actions for uat and testnet#281
willmeister merged 4 commits intomasterfrom
tag_rollback_images

Conversation

@willmeister
Copy link
Copy Markdown

Description

Adding rollback tagging step in github actions for uat and testnet.

This will make it so the rollback image is more easily identifiable if a deployment needs to be rolled back.

Contributing Agreement

@willmeister willmeister merged commit 5886458 into master Sep 24, 2020
@willmeister willmeister deleted the tag_rollback_images branch September 24, 2020 13:49
@willmeister
Copy link
Copy Markdown
Author

Merged without approval because @tynes approved the exact same change in the go-ethereum repo.

snario pushed a commit that referenced this pull request Apr 14, 2021
* Use core-utils where applicable

* Further reduce utils

* Update ovmCREATEEOA.spec.ts

Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>
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>
protolambda added a commit to protolambda/optimism that referenced this pull request May 1, 2022
…ed-up-solidity-ci

Parallelize solidity CI checks
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
This reverts commit 59e02fb3ece7fc249223054944a0b1ff437aa64f.
samlaf pushed a commit to samlaf/optimism that referenced this pull request Aug 20, 2024
shenkeyao referenced this pull request in EspressoSystems/optimism-espresso-integration Mar 1, 2025
Forge started to require the `--broadcast` flag for actually deploying a
contract. Otherwise it will only do a dry-run.

We should really pin our foundry version. But let's wait until we rebase
to the latest upstream, since there have been changes to the overall
setup.

Closes celo-org#278
QuentinI referenced this pull request in EspressoSystems/optimism-espresso-integration Mar 7, 2025
Forge started to require the `--broadcast` flag for actually deploying a
contract. Otherwise it will only do a dry-run.

We should really pin our foundry version. But let's wait until we rebase
to the latest upstream, since there have been changes to the overall
setup.

Closes celo-org#278
agusduha pushed a commit that referenced this pull request Mar 20, 2025
QuentinI referenced this pull request in EspressoSystems/optimism-espresso-integration May 27, 2025
Forge started to require the `--broadcast` flag for actually deploying a
contract. Otherwise it will only do a dry-run.

We should really pin our foundry version. But let's wait until we rebase
to the latest upstream, since there have been changes to the overall
setup.

Closes celo-org#278
dailinsubjam referenced this pull request in EspressoSystems/optimism-espresso-integration Oct 4, 2025
This uses the default fee-currency-directory address from op-geth.
It will fix the issue that the EVM calls into the directoy will fail
when executed on the local devnet.

e2e: use `--broadcast` with `forge create` (#281)

Forge started to require the `--broadcast` flag for actually deploying a
contract. Otherwise it will only do a dry-run.

We should really pin our foundry version. But let's wait until we rebase
to the latest upstream, since there have been changes to the overall
setup.

Closes celo-org#278
theochap pushed a commit that referenced this pull request Jan 15, 2026
### Description

Simplifies signature definitions.
Closes #59.
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