Closed hqdNotional closed 4 months ago
The update_config.sh
script has been enhanced to better manage chain configurations, focusing on RPC and snapshot functionalities. It now includes more robust options for updating environment variables, handling snapshot configurations, and modifying start scripts based on user inputs.
File Path | Change Summary |
---|---|
rpc/scripts/update_config.sh |
Enhanced to manage RPC and snapshot configurations more effectively. |
🐇✨
In the burrow of code, under digital skies,
A script was tweaked, a new dawn lies.
Snapshots and chains, in harmony they spin,
Updates unfurl, let the new begin!
🌟🌿
rpc/scripts/update_config.sh (36)
[style] 8-8: Use $(...) notation instead of legacy backticks `...`. --- [info] 23-23: Double quote to prevent globbing and word splitting. --- [info] 30-30: Double quote to prevent globbing and word splitting. --- [info] 32-32: Not following: ./env.sh was not specified as input (see shellcheck -x). --- [info] 32-32: Double quote to prevent globbing and word splitting. --- [warning] 48-48: chain_name is referenced but not assigned. --- [info] 48-48: Double quote to prevent globbing and word splitting. --- [warning] 59-59: config is referenced but not assigned. --- [info] 75-75: Double quote to prevent globbing and word splitting. --- [warning] 78-78: snapshot_storage_node is referenced but not assigned. --- [info] 89-89: Double quote to prevent globbing and word splitting. --- [info] 89-89: Double quote to prevent globbing and word splitting. --- [info] 89-89: Double quote to prevent globbing and word splitting. --- [info] 92-92: Double quote to prevent globbing and word splitting. --- [warning] 95-95: version is referenced but not assigned. --- [warning] 96-96: daemon_name is referenced but not assigned. --- [warning] 97-97: node_home is referenced but not assigned. --- [warning] 98-98: minimum_gas_prices is referenced but not assigned. --- [warning] 99-99: start_flags is referenced but not assigned. --- [warning] 100-100: db_backend is referenced but not assigned. --- [warning] 101-101: go_version is referenced but not assigned. --- [warning] 102-102: build_script is referenced but not assigned. --- [warning] 105-105: Quote this to prevent word splitting. --- [info] 119-119: Double quote to prevent globbing and word splitting. --- [info] 119-119: Double quote to prevent globbing and word splitting. --- [info] 122-122: Double quote to prevent globbing and word splitting. --- [warning] 130-130: snapshot_node is referenced but not assigned. --- [warning] 133-133: snapshot_prune is referenced but not assigned. --- [warning] 139-139: Quote this to prevent word splitting. --- [error] 157-157: In functions, use return instead of continue. --- [error] 159-159: In functions, use return instead of continue. --- [info] 167-167: Double quote to prevent globbing and word splitting. --- [info] 167-167: Double quote to prevent globbing and word splitting. --- [info] 167-167: Double quote to prevent globbing and word splitting. --- [info] 170-170: Double quote to prevent globbing and word splitting. --- [error] 180-180: In functions, use return instead of continue.
rpc/scripts/update_config.sh (1)
`14-18`: Function `check_chain_status` looks good and serves its purpose effectively.
looks good, best to test it on a rpc and a snapshot node. Remember to backup the env file.
Or backup to .env.bk
instead of removing it.
Summary by CodeRabbit