#70 Adds task ID to Todo comments.

Author: EnchanterIO

cmd/lightchain/init.go

@@ -79,6 +79,8 @@ func newNodeCfgFromCmd(cmd *cobra.Command) (node.Config, setup.Network, error) {
 	// This should be done inside of the `node.Init()` pkg but due to bad design,
 	// creation of new Ethereum node instance from a config accidentally modifies the FS by creating
 	// a keystore dir therefore we have to perform this check as early as possible.
+	//
+	// Todo: After #90 is done, move this code to `Node.Init()`
 	if doesExist, err := fs.DirExists(dataDir); doesExist || err != nil {
 		return node.Config{}, "", fmt.Errorf("unable to initialize lightchain node. %s already exists", dataDir)
 	}

database/node.go

@@ -34,7 +34,7 @@ func NewNode(cfg *Config, consensusAPI conAPI.API, registry *prometheus.Registry
 		return nil, err
 	}
 
-	// Todo: Should be refactored, creating a new instance of a struct SHOULDN'T do any FS changes
+	// Todo: #90 Should be refactored, creating a new instance of a struct SHOULDN'T do any FS changes
 	cfg.GethCfg.EthCfg.NetworkId = cfg.GethCfg.EthCfg.Genesis.Config.ChainID.Uint64()
 	ethereum, err := ethNode.New(&cfg.GethCfg.NodeCfg)
 	if err != nil {

node/node.go

@@ -29,7 +29,7 @@ func NewNode(cfg *Config) (*Node, error) {
 		return nil, err
 	}
 
-	// Todo: Should be refactored, creating a new instance of a struct SHOULDN'T do any FS changes
+	// Todo: #90 Should be refactored, creating a new instance of a struct SHOULDN'T do any FS changes
 	conRPCAPI := conAPI.NewRPCApi(cfg.consensusCfg.RPCListenPort())
 	logger.Debug("Creating new database node instance from config and creates a keystore dir...")
 	dbNode, err := database.NewNode(&cfg.dbCfg, conRPCAPI, prometheusNode.Registry())