Skip to content

ci: split up into parallel jobs for examples and integration tests#726

Closed
tynes wants to merge 16 commits intomasterfrom
ci/multijob
Closed

ci: split up into parallel jobs for examples and integration tests#726
tynes wants to merge 16 commits intomasterfrom
ci/multijob

Conversation

@tynes
Copy link
Copy Markdown
Contributor

@tynes tynes commented May 3, 2021

Description
This PR breaks the system tests up into multiple Github actions jobs so that they can run in parallel. The build happens first and then the other jobs can start that depend on the build.

karlfloersch and others added 16 commits April 26, 2021 13:44
Co-authored-by: smartcontracts <smartcontracts@doge.org>
* feat[contracts]: Use standard RLP transaction format

* fix[l2geth]: Encode transaction as RLP

* fix: Correct gas estimation in integration tests

* fix: Correct gas estimation in integration tests

* Update packages/contracts/contracts/optimistic-ethereum/OVM/predeploys/OVM_SequencerEntrypoint.sol

Co-authored-by: ben-chain <ben@pseudonym.party>

* fix[contracts]: Use isCreate instead of checking target address

* fix[contracts]: Minor optimization in SequencerEntrypoint

* fix[contracts]: Pass max gas to contract call in EOA contract

Co-authored-by: ben-chain <ben@pseudonym.party>
* feat[contracts]: Make ProxyEOA compatible with eip1967

* fix[contracts]: Fix bug introduced by indirect constant

* chore[contracts]: Add changeset

* Update .changeset/old-cycles-invite.md

Co-authored-by: Georgios Konstantopoulos <me@gakonst.com>
* l2geth: remove ovmsigner

Also reduce the diff

Co-authored-by: smartcontracts

* l2geth: add changeset
* l2geth: set rlp encoded tx in txmeta in RPC layer

* l2geth: remove extra setter of txmeta

* chore: add changeset
* feat[contracts]: Make ExecutionManager return data

* fix[l2geth]: fix linting error

* fix[contracts]: Fix build error

* fix[contracts]: fix failing unit tests

* Add changeset

Co-authored-by: Karl Floersch <karl@karlfloersch.com>
* l2geth: api checks for 0 value

* chore: add changeset

* l2geth: remove check for specific gasprice
…619)

* feat[contracts]: Use standard RLP transaction format (#566)

* feat[contracts]: Use standard RLP transaction format

* fix[l2geth]: Encode transaction as RLP

* fix: Correct gas estimation in integration tests

* fix: Correct gas estimation in integration tests

* Update packages/contracts/contracts/optimistic-ethereum/OVM/predeploys/OVM_SequencerEntrypoint.sol

Co-authored-by: ben-chain <ben@pseudonym.party>

* fix[contracts]: Use isCreate instead of checking target address

* fix[contracts]: Minor optimization in SequencerEntrypoint

* fix[contracts]: Pass max gas to contract call in EOA contract

Co-authored-by: ben-chain <ben@pseudonym.party>

* feat[contracts]: Add value transfer to contract account

* fix[contracts]: Tweak transfer logic and add tests

* fix[geth]: Remove logic that rejects value gt 0 txs

* fix: nonce issue in rpc tests

* fix: use correct wallet in rpc value tests

* Update rpc.spec.ts

* cleanup: remove double definition

* chore: add changeset

* chore: add changeset

* tests: delete dead test

* l2geth: log the tx value

* l2geth: pass through zero value at top level

* test: receipt passes

* test: more specifically set balance

Co-authored-by: ben-chain <ben@pseudonym.party>
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>
* dtl: remove legacy decoding

* tests: remove dead test

* chore: add changeset
* Add Goerli v3 deployment

* Add Goerli v3 to README
* dtl: syncing off by one error

* chore: add changeset
* chore: add changeset

* chore: add changeset

* dtl: pass through value field

* core-utils: update and test toRpcString

* lint: fix

* l2geth: parse value fields

* chore: add changeset
* l2geth: prevent fees lower than 21000

* l2geth: remove old check for too high tx gaslimit

* tests: update to use new min gas estimated value

* chore: add changeset

* test: update expected values

* test: remove dead test
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 3, 2021

🦋 Changeset detected

Latest commit: 4c7c0e5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@eth-optimism/data-transport-layer Patch
@eth-optimism/l2geth Patch
@eth-optimism/integration-tests Patch
@eth-optimism/contracts Patch
@eth-optimism/core-utils Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tynes tynes closed this May 3, 2021
OptimismBot pushed a commit that referenced this pull request Dec 4, 2025
github-merge-queue bot pushed a commit that referenced this pull request Dec 5, 2025
…esis (#18490)

* fix: remove todo and update fees depositor address on mainnet (#704)

* chore: set genesis min withdrawal threshold from ten to two eth (#723)

* fix: remove unused errors (#726)
theochap pushed a commit that referenced this pull request Dec 10, 2025
* feat(derive): `BatchProvider` multiplexed stage

* tests

* monolithic mux macro -> viking funeral ⛵🔥

* rebase

* updates

* remove `into_prev`
OptimismBot pushed a commit that referenced this pull request Jan 13, 2026
…esis (#18490)

* fix: remove todo and update fees depositor address on mainnet (#704)

* chore: set genesis min withdrawal threshold from ten to two eth (#723)

* fix: remove unused errors (#726)
maurelian added a commit that referenced this pull request Jan 14, 2026
* chore: update fees depositor address and min withdrawal amount on genesis (#18490)

* fix: remove todo and update fees depositor address on mainnet (#704)

* chore: set genesis min withdrawal threshold from ten to two eth (#723)

* fix: remove unused errors (#726)

* fix: do not use fallback artifacts on PRs branches other than develop

---------

Co-authored-by: Disco <131301107+0xDiscotech@users.noreply.github.com>
Co-authored-by: Maurelian <maurelian@protonmail.ch>
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.

3 participants