Skip to content

fix(contracts): Update Submodule Commit - #1239

Closed
PatStiles wants to merge 2 commits into
stagingfrom
fix/update_contracts_submodule
Closed

fix(contracts): Update Submodule Commit#1239
PatStiles wants to merge 2 commits into
stagingfrom
fix/update_contracts_submodule

Conversation

@PatStiles

Copy link
Copy Markdown
Contributor

@MarcosNicolau caught that when #1107 the commit for some reason was not updated to the latest commit. This pr fixes this by updating to the correct commit in the eigenlayer-middleware submodule.

To Test:

  • Verify you are on the submodule branch
cd contracts/lib/eigenlayer-middleware
git status

You should be on this commit a1801f07e15192ef06db86dfec716eb2b6a4fe90.

  • Deploy the contracts before running a local testnet by first running make anvil_deploy_eigen_contracts, then make anvil_deploy_aligned_contracts, then setting up a running testnet following the testnet guide.
  • Run the whitelist operator script via make operator_whitelist_devnet and remove it via make operator_remove_devnet.
  • Use cast to check the events have been emitted
cast logs --from-block 0 --to-block latest "AddedToWhiteList(address indexed added)" --rpc-url http://localhost:8545 --address 0x851356ae760d987E095750cCeb3bC6014560891C
cast logs --from-block 0 --to-block latest "RemovedFromWhiteList(address indexed added)" --rpc-url http://localhost:8545 --address 0x851356ae760d987E095750cCeb3bC6014560891C

@PatStilesPatStiles changed the title change submodule commitfix(contracts): Update Submodule CommitOct 14, 2024
@github-actions

Copy link
Copy Markdown

Changes to gas cost

Generated at commit: fe00001947e3d0ad893763b2ee67e4c00d5d947c, compared to commit: 07f682c7f5bab0ef2711f7373ba865d4ef9d5258

🧾 Summary (10% most significant diffs)

ContractMethodAvg (+/-)%
AlignedLayerServiceManagercreateNewTask-193 ✅-0.26%

Full diff report 👇
ContractDeployment Cost (+/-)MethodMin (+/-)%Avg (+/-)%Median (+/-)%Max (+/-)%# Calls (+/-)
AlignedLayerServiceManager4,681,797 (0)createNewTask
receive
53,823 (-12)
21,169 (0)
-0.02%
0.00%
73,649 (-193)
44,597 (-186)
-0.26%
-0.42%
73,951 (-30)
45,064 (0)
-0.04%
0.00%
74,809 (+24)
45,064 (0)
+0.03%
0.00%
256 (0)
256 (0)
RegistryCoordinatorHarness5,846,028 (+17,275)

@MarcosNicolauMarcosNicolau left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Everything is working fine!

@uri-99uri-99 mentioned this pull request Oct 17, 2024
@MauroToscano
MauroToscano deleted the fix/update_contracts_submodule branch January 2, 2025 18:24
Sign up for freeto 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

@PatStiles@MarcosNicolau