search

FactomProject / factom

Library for writing Factom clients in go
MIT License
44 stars 19 forks source link

Remove wallet to clear up dependencies + add go.mod #144

Closed WhoSoup closed 4 years ago

WhoSoup commented 4 years ago

The /wallet/ subfolder doesn't really belong into this repo. It's not used by the factom lib and it's not used by factomd. The projects using it are factom-walletd and the enterprise wallet. I move to make the wallet either its own repo, or a sub-repo of one of the two aforementioned projects (probably factom-walletd).

The wallet folder was the cause of all go.mod dependency issues and without it, the go.mod file is relatively small and simple. Before there were hard to fix recursive dependency problems that are now gone.

PaulBernier commented 4 years ago

Great clean up! Can you create an issue for tracking in the factom-walletd repo?