Skip to content

Feature: proxy admin owner on deploy#220

Merged
rymcol merged 3 commits into
canaryfrom
fix/proxy-admin
Dec 18, 2025
Merged

Feature: proxy admin owner on deploy#220
rymcol merged 3 commits into
canaryfrom
fix/proxy-admin

Conversation

@rymcol

@rymcol rymcol commented Dec 17, 2025

Copy link
Copy Markdown
Contributor

Description

Creates and easy way to set the proxy admin owner when deploying

Motivation and Context

Saves the deployer from having to call transfer manually

How Has This Been Tested?

manually

Types of changes (remove all unchecked types)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Style (style only changes)
  • Docs
  • Refactor (code that does not add new functionality nor fixes a bug)

Checklist:

  • If my change requires a change to the documentation, I have updated the documentation accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additions or updates to any deployment scripts to ensure that the protocol is functional (Makefile, Dockerfile, Forge Script, etc.), I have made these changes, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additional test coverage, I have created those tests accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.

@rymcol rymcol requested a review from evchip December 17, 2025 21:27
@rymcol rymcol self-assigned this Dec 17, 2025

@evchip evchip left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks!

@rymcol rymcol merged commit 8c2c432 into canary Dec 18, 2025
4 checks passed
@rymcol rymcol deleted the fix/proxy-admin branch December 18, 2025 00:41
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