archway-network / cli

Develop WASM smart contracts with the Archway network developer CLI
https://docs.archway.io
Apache License 2.0
44 stars 21 forks source link

refactor(config): Remove the `SigningArchwayClient` from the config #231

Closed aelesbao closed 1 year ago

aelesbao commented 1 year ago

Description

The Config domain should handle only reading and writing the config file, with some extra validation logic and nothing else.

Placing the arch3.js clients in this class creates unnecessary coupling and increases the complexity of the code.

Technical Details

Create a builder class that accepts the Config and returns an ArchwayClient or SigningArchwayClient.