Skip to content

Commit

Permalink
Revert the strings.ToLower() for project name as too disruptive
Browse files Browse the repository at this point in the history
  • Loading branch information
rfay committed Jun 6, 2024
1 parent ada5204 commit 4e7c633
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion pkg/ddevapp/ddevapp.go
Original file line number Diff line number Diff line change
Expand Up @@ -2858,7 +2858,7 @@ func GetActiveApp(siteName string) (*DdevApp, error) {

// Lowercase and replace underscores in the site name with hyphens.
func NormalizeProjectName(siteName string) string {
return strings.ToLower(strings.ReplaceAll(siteName, "_", "-"))
return strings.ReplaceAll(siteName, "_", "-")
}

// restoreApp recreates an AppConfig's Name and returns an error
Expand Down
4 changes: 1 addition & 3 deletions pkg/ddevapp/ddevapp_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2988,9 +2988,7 @@ func TestRouterPortsCheck(t *testing.T) {
}

app, err = ddevapp.GetActiveApp(site.Name)
if err != nil {
t.Fatalf("Failed to GetActiveApp(%s), err:%v", site.Name, err)
}
require.NoError(t, err, "Failed to GetActiveApp(%s), err:%v", site.Name, err)
startErr = app.StartAndWait(5)
//nolint: errcheck
defer app.Stop(true, false)
Expand Down

0 comments on commit 4e7c633

Please sign in to comment.