Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove use of deprecated commands #94

Merged
merged 1 commit into from
May 1, 2024
Merged

Conversation

leighmcculloch
Copy link
Member

@leighmcculloch leighmcculloch commented May 1, 2024

What

Remove use of the soroban config commands from tests.

Why

The soroban config commands have been deprecated for a while and replaced with other commands. Soon the deprecated commands will be deleted.

Related to:

@leighmcculloch leighmcculloch requested a review from sreuland May 1, 2024 13:10
@leighmcculloch leighmcculloch marked this pull request as ready for review May 1, 2024 13:10
Copy link
Contributor

@sreuland sreuland left a comment

Choose a reason for hiding this comment

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

lgtm. tried it locally and passed:

$ make \
CORE_GIT_REF=https://github.com/stellar/stellar-core.git#v21.0.0rc1 \
CORE_COMPILE_CONFIGURE_FLAGS="--disable-tests" \
SOROBAN_RPC_GIT_REF=https://github.com/stellar/soroban-rpc.git#main \
SOROBAN_CLI_GIT_REF=https://github.com/stellar/soroban-cli.git#main \
RUST_TOOLCHAIN_VERSION=stable \
RS_XDR_GIT_REF=v21.0.1 \
QUICKSTART_GIT_REF=https://github.com/stellar/quickstart.git#master \
JS_STELLAR_SDK_NPM_VERSION=11.3.0 \
PROTOCOL_VERSION_DEFAULT=21 \
build

$ docker run --rm -t --name e2e_test stellar/system-test:dev \ 
--VerboseOutput true \
--TestFilter "^TestDappDevelop$/^.*$" \
--SorobanExamplesGitHash 782458d31395b0191ac40c9572ac53f13f573aa3

@leighmcculloch leighmcculloch merged commit a10c468 into master May 1, 2024
3 checks passed
@leighmcculloch leighmcculloch deleted the fix-deprecated branch May 1, 2024 22:12
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