Skip to content

Conversation

@maurelian
Copy link
Contributor

@maurelian maurelian commented Oct 15, 2024

TL;DR

Removed special handling for FaultDisputeGame and PermissionedDisputeGame contracts in initialization checks. We no longer need them as the Deploy script labels them properly.

What changed?

  • Removed the restriction on loading initialized slots for FaultDisputeGame and PermissionedDisputeGame in Artifacts.s.sol.
  • Simplified the initialization check in Initializable.t.sol by removing the special case handling for these contracts.

Copy link
Contributor Author

maurelian commented Oct 15, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @maurelian and the rest of your teammates on Graphite Graphite

@maurelian maurelian marked this pull request as ready for review October 15, 2024 16:37
@maurelian maurelian requested a review from a team as a code owner October 15, 2024 16:37
@maurelian maurelian requested review from mbaxter and smartcontracts and removed request for mbaxter October 15, 2024 16:37
@tynes tynes added this pull request to the merge queue Oct 15, 2024
Merged via the queue into develop with commit c8edbe2 Oct 15, 2024
@tynes tynes deleted the load-init-fix branch October 15, 2024 17:52
samlaf pushed a commit to samlaf/optimism that referenced this pull request Nov 10, 2024
…sm#12462)

* test: Remove exceptions for unlabelled fdg contracts

* fix: Unused import
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