Make hugo.toml the new default config name

Note that the old, e.g. config.toml, still works fine. See gohugoio#8979
bep · Sep 18, 2021 · 8783578 · 8783578
1 parent 13ad840
commit 8783578
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 11 deletions.
diff --git a/commands/new_site.go b/commands/new_site.go
@@ -139,7 +139,7 @@ func createConfig(fs *hugofs.Fs, inpath string, kind string) (err error) {
 		return err
 	}
 
-	return helpers.WriteToDisk(filepath.Join(inpath, "config."+kind), &buf, fs.Source)
+	return helpers.WriteToDisk(filepath.Join(inpath, "hugo."+kind), &buf, fs.Source)
 }
 
 func nextStepsText() string {

diff --git a/config/configLoader.go b/config/configLoader.go
@@ -137,7 +137,7 @@ func LoadConfigFromDir(sourceFs afero.Fs, configDir, environment string) (Provid
 
 			var keyPath []string
 
-			if name != "config" {
+			if name != "hugo" && name != "config" {
 				// Can be params.jp, menus.en etc.
 				name, lang := paths.FileAndExtNoDelimiter(name)
 

diff --git a/hugolib/config.go b/hugolib/config.go
@@ -213,7 +213,7 @@ func (d ConfigSourceDescriptor) configFileDir() string {
 
 func (d ConfigSourceDescriptor) configFilenames() []string {
 	if d.Filename == "" {
-		return []string{"config"}
+		return []string{"hugo", "config"}
 	}
 	return strings.Split(d.Filename, ",")
 }

diff --git a/hugolib/config_test.go b/hugolib/config_test.go
@@ -34,25 +34,28 @@ func TestLoadConfig(t *testing.T) {
 
 	c := qt.New(t)
 
-	loadConfig := func(c *qt.C, configContent string, fromDir bool) config.Provider {
+	loadConfig := func(c *qt.C, configFilename, configContent string, fromDir bool) config.Provider {
 		mm := afero.NewMemMapFs()
-		filename := "config.toml"
 		descriptor := ConfigSourceDescriptor{Fs: mm}
 		if fromDir {
-			filename = filepath.Join("config", "_default", filename)
+			configFilename = filepath.Join("config", "_default", configFilename)
 			descriptor.AbsConfigDir = "config"
 		}
-		writeToFs(t, mm, filename, configContent)
+		writeToFs(t, mm, configFilename, configContent)
 		cfg, _, err := LoadConfig(descriptor)
 		c.Assert(err, qt.IsNil)
 		return cfg
 	}
 
 	c.Run("Basic", func(c *qt.C) {
 		c.Parallel()
-		// Add a random config variable for testing.
-		// side = page in Norwegian.
-		cfg := loadConfig(c, `PaginatePath = "side"`, false)
+		cfg := loadConfig(c, "hugo.toml", `PaginatePath = "side"`, false)
+		c.Assert(cfg.GetString("paginatePath"), qt.Equals, "side")
+	})
+
+	c.Run("Basic, old config name", func(c *qt.C) {
+		c.Parallel()
+		cfg := loadConfig(c, "config.toml", `PaginatePath = "side"`, false)
 		c.Assert(cfg.GetString("paginatePath"), qt.Equals, "side")
 	})
 
@@ -64,7 +67,7 @@ func TestLoadConfig(t *testing.T) {
 		}
 		c.Run(testName, func(c *qt.C) {
 			c.Parallel()
-			cfg := loadConfig(c, `[taxonomies]
+			cfg := loadConfig(c, "hugo.toml", `[taxonomies]
 appellation = "appellations"
 vigneron = "vignerons"`, fromDir)