Skip to content

refactor(protocol-contracts): add virtual to staking functions and change private to internal#1694

Merged
melanciani merged 1 commit intomainfrom
melanciani/refactor/protocol-contracts/staking/add_virtual_and_internal
Jan 5, 2026
Merged

refactor(protocol-contracts): add virtual to staking functions and change private to internal#1694
melanciani merged 1 commit intomainfrom
melanciani/refactor/protocol-contracts/staking/add_virtual_and_internal

Conversation

@melanciani
Copy link
Copy Markdown
Contributor

@melanciani melanciani commented Dec 30, 2025

Summary

  • Add virtual modifier to all functions in ProtocolStaking, OperatorRewarder, and OperatorStaking contracts
  • Change private storage getter to internal in both upgradeable staking contracts
  • Update ProtocolStakingSlashingMock to use inherited _getProtocolStakingStorage function

refs https://github.com/zama-ai/fhevm-internal/issues/826

@melanciani melanciani requested a review from a team as a code owner December 30, 2025 17:55
@cla-bot cla-bot bot added the cla-signed label Dec 30, 2025
@melanciani melanciani requested review from arr00 and jatZama December 30, 2025 17:55
@mergify
Copy link
Copy Markdown

mergify bot commented Dec 30, 2025

🧪 CI Insights

Here's what we observed from your CI run for 1f8f325.

🟢 All jobs passed!

But CI Insights is watching 👀

@melanciani melanciani force-pushed the melanciani/refactor/protocol-contracts/staking/add_virtual_and_internal branch from 72714d9 to 649ce64 Compare December 30, 2025 18:02
jatZama
jatZama previously approved these changes Dec 30, 2025
Copy link
Copy Markdown
Member

@jatZama jatZama left a comment

Choose a reason for hiding this comment

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

Great work, looks a lot more practical now.

@melanciani melanciani force-pushed the melanciani/refactor/protocol-contracts/staking/add_virtual_and_internal branch from 649ce64 to 1f8f325 Compare January 5, 2026 12:57
@melanciani melanciani changed the title refactor(protocol-contracts): add virtual to staking functions and change private to internal refactor(protocol-contracts): add virtual to staking functions and change private to internal (refs N-03 Jan 5, 2026
@melanciani melanciani changed the title refactor(protocol-contracts): add virtual to staking functions and change private to internal (refs N-03 refactor(protocol-contracts): add virtual to staking functions and change private to internal Jan 5, 2026
@melanciani melanciani requested a review from jatZama January 5, 2026 13:12
@melanciani melanciani merged commit 53ad63a into main Jan 5, 2026
65 checks passed
@melanciani melanciani deleted the melanciani/refactor/protocol-contracts/staking/add_virtual_and_internal branch January 5, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants