Use *mail.Address instead of custom *Author type

Supports #1586

Author: meatballhat

app.go

@@ -5,6 +5,7 @@ import (
 	"flag"
 	"fmt"
 	"io"
+	"net/mail"
 	"os"
 	"path/filepath"
 	"sort"
@@ -79,7 +80,7 @@ type App struct {
 	// Execute this function when an invalid flag is accessed from the context
 	InvalidFlagAccessHandler InvalidFlagAccessFunc
 	// List of all authors who contributed
-	Authors []*Author
+	Authors []*mail.Address
 	// Copyright of the binary if any
 	Copyright string
 	// Reader reader to write input to (useful for tests)
@@ -434,22 +435,6 @@ func runFlagActions(c *Context, fs []Flag) error {
 	return nil
 }
 
-// Author represents someone who has contributed to a cli project.
-type Author struct {
-	Name  string // The Authors name
-	Email string // The Authors email
-}
-
-// String makes Author comply to the Stringer interface, to allow an easy print in the templating process
-func (a *Author) String() string {
-	e := ""
-	if a.Email != "" {
-		e = " <" + a.Email + ">"
-	}
-
-	return fmt.Sprintf("%v%v", a.Name, e)
-}
-
 func checkStringSliceIncludes(want string, sSlice []string) bool {
 	found := false
 	for _, s := range sSlice {

app_test.go

@@ -6,6 +6,7 @@ import (
 	"flag"
 	"fmt"
 	"io"
+	"net/mail"
 	"os"
 	"reflect"
 	"strconv"
@@ -45,7 +46,7 @@ func ExampleApp_Run() {
 			return nil
 		},
 		UsageText: "app [first_arg] [second_arg]",
-		Authors:   []*Author{{Name: "Oliver Allen", Email: "[email protected]"}},
+		Authors:   []*mail.Address{{Name: "Oliver Allen", Address: "[email protected]"}},
 	}
 
 	app.Run(os.Args)
@@ -100,9 +101,9 @@ func ExampleApp_Run_appHelp() {
 		Name:        "greet",
 		Version:     "0.1.0",
 		Description: "This is how we describe greet the app",
-		Authors: []*Author{
-			{Name: "Harrison", Email: "[email protected]"},
-			{Name: "Oliver Allen", Email: "[email protected]"},
+		Authors: []*mail.Address{
+			{Name: "Harrison", Address: "[email protected]"},
+			{Name: "Oliver Allen", Address: "[email protected]"},
 		},
 		Flags: []Flag{
 			&StringFlag{Name: "name", Value: "bob", Usage: "a name to say"},
@@ -135,8 +136,8 @@ func ExampleApp_Run_appHelp() {
 	//    This is how we describe greet the app
 	//
 	// AUTHORS:
-	//    Harrison <[email protected]>
-	//    Oliver Allen <[email protected]>
+	//    "Harrison" <[email protected]>
+	//    "Oliver Allen" <[email protected]>
 	//
 	// COMMANDS:
 	//    describeit, d  use it to see a description

docs_test.go

@@ -5,6 +5,7 @@ package cli
 
 import (
 	"errors"
+	"net/mail"
 	"testing"
 )
 
@@ -49,7 +50,7 @@ func TestToMarkdownNoCommands(t *testing.T) {
 func TestToMarkdownNoAuthors(t *testing.T) {
 	// Given
 	app := testApp()
-	app.Authors = []*Author{}
+	app.Authors = []*mail.Address{}
 
 	// When
 	res, err := app.ToMarkdown()

fish_test.go

@@ -2,6 +2,7 @@ package cli
 
 import (
 	"bytes"
+	"net/mail"
 	"os"
 	"testing"
 )
@@ -127,9 +128,9 @@ Should be a part of the same code block
 	app.UsageText = "app [first_arg] [second_arg]"
 	app.Description = `Description of the application.`
 	app.Usage = "Some app"
-	app.Authors = []*Author{
-		{Name: "Harrison", Email: "[email protected]"},
-		{Name: "Oliver Allen", Email: "[email protected]"},
+	app.Authors = []*mail.Address{
+		{Name: "Harrison", Address: "[email protected]"},
+		{Name: "Oliver Allen", Address: "[email protected]"},
 	}
 	return app
 }

godoc-current.txt

@@ -294,7 +294,7 @@ type App struct {
 	// Execute this function when an invalid flag is accessed from the context
 	InvalidFlagAccessHandler InvalidFlagAccessFunc
 	// List of all authors who contributed
-	Authors []*Author
+	Authors []*mail.Address
 	// Copyright of the binary if any
 	Copyright string
 	// Reader reader to write input to (useful for tests)
@@ -399,16 +399,6 @@ type Args interface {
 	Slice() []string
 }
 
-type Author struct {
-	Name  string // The Authors name
-	Email string // The Authors email
-}
-    Author represents someone who has contributed to a cli project.
-
-func (a *Author) String() string
-    String makes Author comply to the Stringer interface, to allow an easy print
-    in the templating process
-
 type BeforeFunc func(*Context) error
     BeforeFunc is an action to execute before any subcommands are run, but after
     the context is ready if a non-nil error is returned, no subcommands are run

testdata/godoc-v3.x.txt

@@ -294,7 +294,7 @@ type App struct {
 	// Execute this function when an invalid flag is accessed from the context
 	InvalidFlagAccessHandler InvalidFlagAccessFunc
 	// List of all authors who contributed
-	Authors []*Author
+	Authors []*mail.Address
 	// Copyright of the binary if any
 	Copyright string
 	// Reader reader to write input to (useful for tests)
@@ -399,16 +399,6 @@ type Args interface {
 	Slice() []string
 }
 
-type Author struct {
-	Name  string // The Authors name
-	Email string // The Authors email
-}
-    Author represents someone who has contributed to a cli project.
-
-func (a *Author) String() string
-    String makes Author comply to the Stringer interface, to allow an easy print
-    in the templating process
-
 type BeforeFunc func(*Context) error
     BeforeFunc is an action to execute before any subcommands are run, but after
     the context is ready if a non-nil error is returned, no subcommands are run