cleanup: rename MakeDBConf to NewDBConf to more clearly represent what it's actually doing. No functional changes.

cmd_create.go

@@ -30,7 +30,7 @@ func createRun(cmd *Command, args ...string) {
 		}
 	}
 
-	conf, err := MakeDBConf()
+	conf, err := NewDBConf()
 	if err != nil {
 		log.Fatal(err)
 	}

cmd_down.go

@@ -16,7 +16,7 @@ var downCmd = &Command{
 
 func downRun(cmd *Command, args ...string) {
 
-	conf, err := MakeDBConf()
+	conf, err := NewDBConf()
 	if err != nil {
 		log.Fatal(err)
 	}

cmd_status.go

@@ -22,7 +22,7 @@ type StatusData struct {
 
 func statusRun(cmd *Command, args ...string) {
 
-	conf, err := MakeDBConf()
+	conf, err := NewDBConf()
 	if err != nil {
 		log.Fatal(err)
 	}

cmd_up.go

@@ -15,7 +15,7 @@ var upCmd = &Command{
 
 func upRun(cmd *Command, args ...string) {
 
-	conf, err := MakeDBConf()
+	conf, err := NewDBConf()
 	if err != nil {
 		log.Fatal(err)
 	}

dbconf.go

@@ -29,12 +29,12 @@ type DBConf struct {
 }
 
 // default helper - makes a DBConf from the dbPath and dbEnv flags
-func MakeDBConf() (*DBConf, error) {
-	return makeDBConfDetails(*dbPath, *dbEnv)
+func NewDBConf() (*DBConf, error) {
+	return newDBConfDetails(*dbPath, *dbEnv)
 }
 
 // extract configuration details from the given file
-func makeDBConfDetails(p, env string) (*DBConf, error) {
+func newDBConfDetails(p, env string) (*DBConf, error) {
 
 	cfgFile := filepath.Join(p, "dbconf.yml")
 

dbconf_test.go

@@ -7,7 +7,7 @@ import (
 
 func TestBasics(t *testing.T) {
 
-	dbconf, err := makeDBConfDetails("db-sample", "test")
+	dbconf, err := newDBConfDetails("db-sample", "test")
 	if err != nil {
 		t.Error("couldn't create DBConf")
 	}
@@ -24,7 +24,7 @@ func TestBasics(t *testing.T) {
 
 func TestImportOverride(t *testing.T) {
 
-	dbconf, err := makeDBConfDetails("db-sample", "customimport")
+	dbconf, err := newDBConfDetails("db-sample", "customimport")
 	if err != nil {
 		t.Error("couldn't create DBConf")
 	}