stellar-deprecated / kelp

Kelp is a free and open-source trading bot for the Stellar DEX and 100+ centralized exchanges
https://kelpbot.io
Other
1.1k stars 263 forks source link

Upgrade to use stellar/go@horizonclient-v8.0.0 #748

Closed leighmcculloch closed 2 years ago

leighmcculloch commented 2 years ago

This PR is a draft and is likely incomplete.

I've taken a time-boxed pass at upgrading the stellar/go dependency from horizonclient-v3.0.0 to horizonclient-v8.0.0. I mostly just wanted to explore, understand, and validate assumptions about how Kelp could be upgraded, assuming that the upgrade path would not be smooth.

The intention was to understand, not necessarily deliver an upgrade.

I've opened this draft PR to have this change run on Kelp's CI, to see if at least from CI's perspective if this idea for how to make the upgrade happen, has substance or creates more problems than it solves.

The approach I've taken is to:

Maybe fix #741