Fix envars with numbers.

Fixes #138.
alecthomas · Aug 29, 2016 · e9044be · e9044be
1 parent e590021
commit e9044be
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/app.go b/app.go
@@ -13,7 +13,7 @@ var (
 )
 
 var (
-	envarTransformRegexp = regexp.MustCompile(`[^a-zA-Z_]+`)
+	envarTransformRegexp = regexp.MustCompile(`[^a-zA-Z0-9_]+`)
 )
 
 type ApplicationValidator func(*Application) error

diff --git a/app_test.go b/app_test.go
@@ -218,10 +218,13 @@ func TestDefaultEnvars(t *testing.T) {
 	f0.Bool()
 	f1 := a.Flag("some-other-flag", "").NoEnvar()
 	f1.Bool()
+	f2 := a.Flag("a-1-flag", "")
+	f2.Bool()
 	_, err := a.Parse([]string{})
 	assert.NoError(t, err)
 	assert.Equal(t, "SOME_APP_SOME_FLAG", f0.envar)
 	assert.Equal(t, "", f1.envar)
+	assert.Equal(t, "SOME_APP_A_1_FLAG", f2.envar)
 }
 
 func TestBashCompletionOptionsWithEmptyApp(t *testing.T) {