Skip to content

fix: missing solana-signer extration from solana-sdk#3995

Merged
jacobcreech merged 12 commits intosolana-foundation:masterfrom
Otter-0x4ka5h:pr-3896
Nov 20, 2025
Merged

fix: missing solana-signer extration from solana-sdk#3995
jacobcreech merged 12 commits intosolana-foundation:masterfrom
Otter-0x4ka5h:pr-3896

Conversation

@Otter-0x4ka5h
Copy link
Copy Markdown
Contributor

@Otter-0x4ka5h Otter-0x4ka5h commented Oct 17, 2025

Closes #3884

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 17, 2025

@pratikbuilds is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@nutafrost nutafrost moved this to Security Review Done in Anchor 1.0 Oct 17, 2025
@nutafrost nutafrost moved this from Security Review Done to Security Review Required in Anchor 1.0 Oct 17, 2025
@nutafrost nutafrost removed the Merge label Oct 17, 2025
@nutafrost nutafrost assigned jamie-osec and unassigned jacobcreech Oct 17, 2025
@Aursen Aursen requested review from Aursen and removed request for Aursen October 17, 2025 09:58
Copy link
Copy Markdown
Collaborator

@jamie-osec jamie-osec left a comment

Choose a reason for hiding this comment

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

Remaining imports:

  • client/src/blocking.rs:13
  • client/src/nonblocking.rs:9

Also some docs examples which imo should be fixed for best practice:

  • docs/content/docs/features/declare-program.mdx:596
  • docs/content/docs/clients/rust.mdx:176
  • client/src/lib.rs:15

@jamie-osec
Copy link
Copy Markdown
Collaborator

Additionally, as a drive-by fix, I would suggest making the solana-signer dep a workspace dependency to make synchronizing version updates easier (#3892)

@swaroop-osec
Copy link
Copy Markdown
Collaborator

@Otter-0x4ka5h Please fix the failed tests.

@jamie-osec jamie-osec added fix Bug fix PR Merge labels Oct 28, 2025
@nutafrost nutafrost moved this from Security Review Required to Security Review Done in Anchor 1.0 Oct 28, 2025
@jacobcreech jacobcreech merged commit c7b8bb4 into solana-foundation:master Nov 20, 2025
54 of 55 checks passed
@github-project-automation github-project-automation bot moved this from Security Review Done to Done in Anchor 1.0 Nov 20, 2025
Otter-0x4ka5h added a commit to Otter-0x4ka5h/anchor that referenced this pull request Mar 25, 2026
…on#3995)

* refactor: extract solana-signer from solana-sdk in anchor-client and anchor-cli

* Revert "Revert "feat: tests""

This reverts commit c8cf603.

* requested changes

* fix: docs

* fix: cargo.toml

* fix: ci tests

* fix: ci tests

* add dependency

* add dependency

* add dependency

* fix: remove from workspace

---------

Co-authored-by: 0xPratik <sariapratik@gmail.com>
Co-authored-by: Akash Thota <your.email@example.com>
Co-authored-by: Akash Thota <0x4ka5h@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fix PR Merge

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

split out deps into multiple crates; saner architecture

7 participants