Skip to content

Changed number of signers from 3 to 2#192

Merged
dkcumming merged 1 commit intoproofsfrom
mk/num-signers-2
Apr 10, 2026
Merged

Changed number of signers from 3 to 2#192
dkcumming merged 1 commit intoproofsfrom
mk/num-signers-2

Conversation

@mariaKt
Copy link
Copy Markdown

@mariaKt mariaKt commented Apr 10, 2026

This PR changes the number of signers for the previously added test_process_revoke_multisig_3sig and test_process_withdraw_excess_lamports_multisig_multisig_3sig from 3 to 2.

Changes:

  • test_process_revoke_multisig_3sig -> test_process_revoke_multisig_2sig ([AccountInfo; 5] -> [AccountInfo; 4], 2
    tx_signers)
  • test_process_withdraw_excess_lamports_multisig_multisig_3sig ->
    test_process_withdraw_excess_lamports_multisig_multisig_2sig ([AccountInfo; 6] -> [AccountInfo; 5], 2 tx_signers)
  • Dispatch updated in both p-token and spl-token entrypoints (base + 2 instead of base + MAX_SIGNERS)
  • proofs.md entries renamed

@mariaKt mariaKt requested a review from dkcumming April 10, 2026 14:28
Copy link
Copy Markdown
Collaborator

@dkcumming dkcumming left a comment

Choose a reason for hiding this comment

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

lgtm

@dkcumming dkcumming merged commit 85016e7 into proofs Apr 10, 2026
2 checks passed
@mariaKt mariaKt deleted the mk/num-signers-2 branch April 10, 2026 14:42
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