celestiaorg / celestiaorg/celestia-node

Consider separating monolithic config into separate configs, allowing `nodebuilder.Store` to move to `nodebuilder/node.Store`

Open
#1,144 0 comments 0 reactions 0 assignees View on GitHub
area:config area:node
Dominant language
Go
Stars
996
Forks
1.1k
Avg merge
1d 6h
Merged PRs (30d)
34

Description

Move nodebuilder.Store to nodebuilder/node
* There is a dependency cycle with it, and the cycle just reveals the symptom of another problem - monolithic Config
* The Store has to be changed to allow storing Configs independently
```
Config(module string) (interface, error)
PutConfig(module string. interface) (error)
```
In both cases, the Config access can go either to the same file or different. depends on the implementation of the Store

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.