Skip to content

Update address check scripts#647

Merged
rossgalloway merged 4 commits intoyearn:masterfrom
engn33r:update-address-check-scripts
Feb 24, 2026
Merged

Update address check scripts#647
rossgalloway merged 4 commits intoyearn:masterfrom
engn33r:update-address-check-scripts

Conversation

@engn33r
Copy link
Collaborator

@engn33r engn33r commented Feb 24, 2026

No description provided.

@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
yearn-docs Ready Ready Preview, Comment Feb 24, 2026 9:46pm

Request Review

@rossgalloway
Copy link
Collaborator

I pushed a new commit to your branch:
goal is to make the check more trustworthy and more robust.

Before, it looked up the multisig address from a local constant, which could be stale. Now it uses the live on-chain value from the role manager, so if governance rotates the multisig, the check follows that change automatically.

It also now handles failures gracefully. If the multisig address is missing/invalid, or if reading Safe owners fails, the system records a failed check with a useful error message instead of crashing the whole address-check job.

And there’s a new consistency check that compares the stored daddy address constant against the on-chain daddy address, so stale config gets flagged directly.

@rossgalloway rossgalloway merged commit ad68502 into yearn:master Feb 24, 2026
4 checks passed
@engn33r engn33r deleted the update-address-check-scripts branch February 24, 2026 22:43
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