Merge pull request #96 from metalmatze/fix-flags-required

Fix flags with defaults that aren't required anymore
metalmatze · Jan 11, 2020 · 65ec8cd · 65ec8cd
2 parents fd9c008 + 2f22387
commit 65ec8cd
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/cmd/alertmanager-bot/main.go b/cmd/alertmanager-bot/main.go
@@ -71,7 +71,6 @@ func main() {
 	a.HelpFlag.Short('h')
 
 	a.Flag("alertmanager.url", "The URL that's used to connect to the alertmanager").
-		Required().
 		Envar("ALERTMANAGER_URL").
 		Default("http://localhost:9093/").
 		URLVar(&config.alertmanager)
@@ -87,7 +86,6 @@ func main() {
 		URLVar(&config.consul)
 
 	a.Flag("listen.addr", "The address the alertmanager-bot listens on for incoming webhooks").
-		Required().
 		Envar("LISTEN_ADDR").
 		Default("0.0.0.0:8080").
 		StringVar(&config.listenAddr)