Skip to content

Fix remove validator Talos action - #2968

Merged
sparrowDom merged 3 commits into
masterfrom
nicka/fix-remove-validator
Aug 10, 2026
Merged

Fix remove validator Talos action#2968
sparrowDom merged 3 commits into
masterfrom
nicka/fix-remove-validator

Conversation

@naddison36

@naddison36 naddison36 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes Talos actions that referenced Hardhat-provided global ethers and hre objects, which are unavailable in the standalone Talos runtime.

  • Use the action signer’s provider to resolve the chain ID in removeValidator.
  • Remove the unused hre.network argument from the SSV cluster lookup.
  • Use the signer’s provider when checking strategy ETH balances for automatic validator deposits and withdrawals.

Testing

  • Ran Prettier on the changed JavaScript files.
  • Ran focused ESLint checks.
  • Ran git diff --check.
  • Verified all registered Talos action modules load successfully.

@sparrowDom sparrowDom left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@sparrowDom
sparrowDom merged commit 6b0fd24 into master Aug 10, 2026
11 checks passed
@sparrowDom
sparrowDom deleted the nicka/fix-remove-validator branch August 10, 2026 09:58
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