Skip to content

Commit 50fb38a

Browse files
committed
update snfoundry uninstall instructions
add #2 todo
1 parent 20ac8da commit 50fb38a

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

starknetup.sh

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,9 @@
44
set -eu
55

66
ASDF_INSTALL_DOCS="https://asdf-vm.com/guide/getting-started.html"
7-
SCARB_UNINSTALL_DOCS="https://docs.swmansion.com/scarb/download#uninstall"
8-
STARKNET_FOUNDRY_UNINSTALL_DOCS="PENDING"
7+
SCARB_UNINSTALL_INSTRUCTIONS="For uninstallation instructions, refer to https://docs.swmansion.com/scarb/download#uninstall"
8+
# TODO(#2): Link snfoundry uninstall docs once they are available
9+
STARKNET_FOUNDRY_UNINSTALL_INSTRUCTIONS="Consider removing snforge and sncast binaries from $HOME/.local/bin"
910
SCRIPT_VERSION="0.1.0"
1011
DEFAULT_ASDF_VERSION="v0.14.1"
1112

@@ -40,12 +41,12 @@ main() {
4041
done
4142

4243
assert_dependencies
43-
assert_not_installed "scarb" $SCARB_UNINSTALL_DOCS
44+
assert_not_installed "scarb" "$SCARB_UNINSTALL_INSTRUCTIONS"
4445
install_latest_asdf_plugin "scarb"
4546
install_latest_version "scarb"
4647
set_global_latest_version "scarb"
4748

48-
assert_not_installed "starknet-foundry" $STARKNET_FOUNDRY_UNINSTALL_DOCS
49+
assert_not_installed "starknet-foundry" "$STARKNET_FOUNDRY_UNINSTALL_INSTRUCTIONS"
4950
install_latest_asdf_plugin "starknet-foundry"
5051

5152
# Reinstall to ensure the latest version of USC is installed
@@ -90,11 +91,11 @@ assert_dependencies() {
9091

9192
assert_not_installed() {
9293
local tool="$1"
93-
local uninstall_docs_url="$2"
94+
local uninstall_instructions="$2"
9495

9596
if ! asdf which "$tool" >/dev/null 2>&1; then
9697
if check_cmd "$tool"; then
97-
err "$tool is already installed outside of asdf. Please uninstall it and re-run this script. For uninstallation instructions, refer to $uninstall_docs_url."
98+
err "$tool is already installed outside of asdf. Please uninstall it and re-run this script. $uninstall_instructions"
9899
fi
99100
fi
100101
}

0 commit comments

Comments
 (0)