Skip to content

jovian: remove feature toggles#17978

Merged
geoknee merged 8 commits intodevelopfrom
gk/toggle-off
Dec 8, 2025
Merged

jovian: remove feature toggles#17978
geoknee merged 8 commits intodevelopfrom
gk/toggle-off

Conversation

@geoknee
Copy link
Contributor

@geoknee geoknee commented Oct 22, 2025

scope is now fixed.

depends on ethereum-optimism/op-geth#703

@geoknee geoknee requested review from a team as code owners October 22, 2025 11:04
@geoknee geoknee requested a review from sebastianst October 22, 2025 11:04
@geoknee geoknee added the H-jovian Hardfork: change is planned for jovian upgrade label Oct 22, 2025
@geoknee geoknee requested a review from a team as a code owner October 22, 2025 11:12
@geoknee geoknee requested a review from stevennevins October 22, 2025 11:12
@geoknee geoknee force-pushed the gk/toggle-off branch 2 times, most recently from ef83bd7 to 5aff201 Compare October 22, 2025 13:45
@codecov
Copy link

codecov bot commented Oct 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.58%. Comparing base (f6a0ea9) to head (67299de).
⚠️ Report is 2 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (f6a0ea9) and HEAD (67299de). Click for more details.

HEAD has 23 uploads less than BASE
Flag BASE (f6a0ea9) HEAD (67299de)
cannon-go-tests-64 12 1
contracts-bedrock-tests 12 0
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #17978      +/-   ##
===========================================
- Coverage    75.75%   66.58%   -9.18%     
===========================================
  Files          189       55     -134     
  Lines        11228     4031    -7197     
===========================================
- Hits          8506     2684    -5822     
+ Misses        2576     1203    -1373     
+ Partials       146      144       -2     
Flag Coverage Δ
cannon-go-tests-64 66.58% <ø> (-0.82%) ⬇️
contracts-bedrock-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 139 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@geoknee geoknee force-pushed the gk/toggle-off branch 2 times, most recently from 3204818 to 4954a4b Compare October 23, 2025 07:21
@geoknee geoknee added the M-do-not-merge Meta: Do not merge label Oct 24, 2025
@opgitgovernance opgitgovernance added the S-stale Status: Will be closed unless there is activity label Nov 7, 2025
@opgitgovernance
Copy link
Contributor

This pr has been automatically marked as stale and will be closed in 5 days if no updates

@geoknee geoknee removed the S-stale Status: Will be closed unless there is activity label Nov 7, 2025
@geoknee geoknee removed the M-do-not-merge Meta: Do not merge label Nov 20, 2025
@geoknee geoknee enabled auto-merge December 8, 2025 09:52
@geoknee geoknee added this pull request to the merge queue Dec 8, 2025
Merged via the queue into develop with commit e355cc4 Dec 8, 2025
89 checks passed
@geoknee geoknee deleted the gk/toggle-off branch December 8, 2025 10:28
0xniha pushed a commit to defi-wonderland/optimism that referenced this pull request Dec 8, 2025
* jovian: remove feature toggles

scope is now fixed

* Updated op-geth to v1.101604.0-synctest.0.0.20251120150812-e50f80a16afc

* lint

* just update-op-geth f48f382

* Use Jovian in test error message

* just update-op-geth ba6bdcfef42341fe2b5ce124c31ff2d6b264e9e4
github-merge-queue bot pushed a commit that referenced this pull request Dec 11, 2025
* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* refactor: change disputeGameConfigs order in _loadFullConfig

* fix: comment slash rules

* feat: add opcmv2-cgt matrix in CI

* fix: enable cgt overrides on OPCMv2 (#724)

* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides

* fix: add extra instruction overrides for useCustomGasToken on OPCMv2 (#725)

* chore: link TODO to issue (#727)

* fix: opcm2 revert upgrade cgt (#728)

* fix: add check enabled CGT during upgrades

* test: set CGT false by default during forked tests

* chore: expand comment on OPCMv2 regarding CGT

* chore: pre-pr ready

* refactor: remove unnecessary check for CGT feature (#735)

* fix: cgt upgrade in opcm v2 and semver (#738)

* fix: Full checkout for FFI build (#18527)

* chore(op-acceptance-tests): delete very old logs (#18529)

* chore(op-acceptance-tests): op-acceptor v3.8.0 (#18530)

* jovian: remove feature toggles (#17978)

* jovian: remove feature toggles

scope is now fixed

* Updated op-geth to v1.101604.0-synctest.0.0.20251120150812-e50f80a16afc

* lint

* just update-op-geth f48f382

* Use Jovian in test error message

* just update-op-geth ba6bdcfef42341fe2b5ce124c31ff2d6b264e9e4

* chore(ai-eng): add ReinitializableBase test to exclusion list (#18531)

* feat: have OPCM upgrade allowances be upgrade specific (#18462)

* feat: have OPCM upgrade allowances be upgrade specific

Updates the OPCMv2 check for allowed extra instructions to be
specific to releases. When release versions are bumped, the
allowances become automatically invalid and would reveal
anywhere in the codebase where the allowance is being used.

* fix: semver lock

* fix: emit instruction key in error

* fix: better semantics

* docs: proper OPCM versioning policy

* fix: final version tweaks

* fix(op-acceptance-test): flake-shake; empty slack notifications. (#18542)

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* feat: add cgt to opcmv2

* feat: add cgt to opcmv2

* fix: change semver

* refactor: change disputeGameConfigs order in _loadFullConfig

* fix: enable cgt overrides on OPCMv2 (#724)

* fix: check CGT is not enabled before enabling it

* fix: enable CGT overrides

* fix: opcm2 revert upgrade cgt (#728)

* fix: add check enabled CGT during upgrades

* test: set CGT false by default during forked tests

* chore: expand comment on OPCMv2 regarding CGT

* fix: cgt upgrade in opcm v2 and semver (#738)

* fix: merge conflicts

* fix: pre-pr

* fix: add missing isMatchingInstruction for cgt in false

* refactor: opcm2 extra instruction keymatch (#747)

* refactor: add helper function to match instruction by key only

* chore: pre-pr ready

* test: refactor key matching test

* test: add test suite for IsMatchingKey helper function

* chore: pre-pr ready

* fix: opcmv2 semver

---------

Co-authored-by: Flux <175354924+0xiamflux@users.noreply.github.com>
Co-authored-by: Ján Jakub Naništa <jan.jakub.nanista@gmail.com>
Co-authored-by: Stefano Charissis <stefano@oplabs.co>
Co-authored-by: George Knee <georgeknee@googlemail.com>
Co-authored-by: Ariel Diaz <65925295+aliersh@users.noreply.github.com>
Co-authored-by: smartcontracts <14298799+smartcontracts@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

H-jovian Hardfork: change is planned for jovian upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments