Fix urfavecli.LogFlags

Now that urfave/cli#1210 has landed.

Change-Id: If8bffcb2e470e2d7285a4d2c8f1672650f0ec43f
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/363156
Auto-Submit: Joe Gregorio <jcgregorio@google.com>
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>

go/urfavecli/urfavecli.go

@@ -20,11 +20,14 @@ import (
 //          },
 //      },
 func LogFlags(cliContext *cli.Context) {
+	for _, flag := range cliContext.App.Flags {
+		name := flag.Names()[0]
+		sklog.Infof("App Flags: --%s=%v", name, cliContext.Value(name))
+	}
 	for _, flag := range cliContext.Command.Flags {
 		name := flag.Names()[0]
-		sklog.Infof("Flags: --%s=%v", name, cliContext.Value(name))
+		sklog.Infof("Command Flags: --%s=%v", name, cliContext.Value(name))
 	}
-
 }
 
 // MarkdownDocTemplate is a common template used to format commands as Markdown.

go/urfavecli/urfavecli_test.go

@@ -161,6 +161,7 @@ func TestLogFlags(t *testing.T) {
 	}
 
 	expected := []string{
+		" --help=false",
 		" --boolNotPassedIn=false",
 		" --bool=true",
 		" --duration=24s",