Skip to content

[chore]sdk-examples: fix broken py-stellar-base imports#2545

Open
ElliotFriend wants to merge 1 commit into
mainfrom
chore/sdk-examples-python-14.1.1-audit
Open

[chore]sdk-examples: fix broken py-stellar-base imports#2545
ElliotFriend wants to merge 1 commit into
mainfrom
chore/sdk-examples-python-14.1.1-audit

Conversation

@ElliotFriend

Copy link
Copy Markdown
Contributor

Correctness audit against py-stellar-base v14.1.1

Standing-correctness audit against py-stellar-base 14.1.1:

learn/migrate/evm/smart-contract-deployment.mdx imported from the
pre-v9 `stellar_sdk.soroban.*` sub-package, which no longer exists
(the module layout is flat: `soroban_server`, `soroban_rpc`).
`SorobanServer` is re-exported at the top level, so the import is now
`from stellar_sdk import SorobanServer`. The second import
(`TransactionStatus` from `stellar_sdk.soroban.soroban_rpc`) referenced
a symbol that no longer exists under that name and was unused in the
snippet, so it was removed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 2, 2026 13:53

Copilot AI 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.

Pull request overview

This PR updates the Python example in the EVM migration documentation to align with py-stellar-base / stellar-sdk v14.1.1 by fixing the import path for SorobanServer.

Changes:

  • Replace deprecated/broken imports from stellar_sdk.soroban / stellar_sdk.soroban.soroban_rpc with the v14.1.1-compatible from stellar_sdk import SorobanServer.
  • Remove the unused TransactionStatus import (it is not referenced elsewhere in the snippet).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ElliotFriend ElliotFriend changed the title chore(sdk-examples): fix broken py-stellar-base imports [chore]sdk-examples: fix broken py-stellar-base imports Jul 2, 2026
@stellar-jenkins-ci

Copy link
Copy Markdown

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.

3 participants