Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: from scratch scripts #335

Open
wants to merge 49 commits into
base: feat/slashing-release-branch
Choose a base branch
from

Conversation

stevennevins
Copy link
Collaborator

No description provided.

@stevennevins stevennevins changed the base branch from mainnet to feat/slashing-release-branch December 13, 2024 14:12
stevennevins and others added 9 commits December 13, 2024 09:24
* fix: remove registerOperatorToOperatorSet interface function

* fix: update interface functions with alm interface

* fix: operator set strategies in ALM

* chore: remove todo

* fix: add strategies by stake registry

* fix: params for deregister

* chore: remove old migration functions

* chore: check quorum exists before setting params

* fix: deregister flow for operator set quorums
* feat: custom require errors for registry coordinator

* ci: update the ci to use the correct compiler settings
@stevennevins stevennevins changed the base branch from feat/slashing-release-branch to fix/slashing-release-branch-1 December 13, 2024 17:08
Base automatically changed from fix/slashing-release-branch-1 to feat/slashing-release-branch December 16, 2024 14:25
.github/workflows/tests.yml Outdated Show resolved Hide resolved
script/DeployMiddleware.s.sol Show resolved Hide resolved
script/DeployMiddleware.s.sol Outdated Show resolved Hide resolved
script/config/17000-preprod.json Show resolved Hide resolved
script/utils/MiddlewareDeploymentLib.sol Show resolved Hide resolved
test/unit/ZeusDeploymentTests.t.sol Show resolved Hide resolved
struct DeploymentData {
address delegationManager;
address avsDirectory;
address allocationManager;
Copy link
Collaborator

Choose a reason for hiding this comment

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

AllocationManagerConfig seems to be missing

addrs.avsDirectoryImpl,
addrs.strategyManagerImpl,
addrs.strategyFactoryImpl
) = _deployMainImplementations(result, config);
Copy link
Collaborator

Choose a reason for hiding this comment

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

ALM also seems to be missing from _deployMainImplementations and _deployRemainingImplementations

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