search

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.09k stars 262 forks source link

Make local development and running simpler #757

Closed leighmcculloch closed 2 years ago

leighmcculloch commented 2 years ago

What

Don't require build information when running locally during development.

Why

When developing locally it is useful to be able to run kelp as is with simple commands like go run .. That isn't possible today because kelp requires a variety of build parameters to be specified. That's inconvenient for local development, and it is only really necessary to have all these build parameters included when releasing.

leighmcculloch commented 2 years ago

@nikhilsaraf The integration test is failing with this error:

2022/02/11 01:31:56 could not get last trade cursor from exchangeShim: error while fetching latest trade cursor in SDEX: horizon error: "Resource Missing" - check horizon.Error.Problem for more information (quoteAssetError: asset 'CITEST:GBMMZMK2DC4FFP4CAI6KCVNCQ7WLO5A7DQU7EC7WGHRDQBZB763X4OQI' did not exist) 2022/02/11 01:31:56 metric - not sending event metric of type 'delete_offers' because metrics are disabled

I don't think that is a result of the changes I'm making here. How do we fix this?

nikhilsaraf commented 2 years ago

The secret keys for that are in the circle ci file. I manually initialize it and issue assets to the trader account when the testnet is reset