Skip to content

Commit

Permalink
Merge pull request #205 from atc0005/fix-goconst-linting-errors
Browse files Browse the repository at this point in the history
Fix goconst linting errors
  • Loading branch information
atc0005 authored Nov 16, 2023
2 parents ec4c163 + 24a2457 commit cfc11c1
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
4 changes: 4 additions & 0 deletions internal/config/constants.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ const (
myAppURL string = "https://github.com/atc0005/tsm-pass"
)

// shorthandFlagSuffix is appended to short flag help text to emphasize that
// the flag is a shorthand version of a longer flag.
const shorthandFlagSuffix = " (shorthand)"

// Help text used with flags
const (
versionFlagHelp string = "Whether to display application version and then immediately exit application."
Expand Down
8 changes: 4 additions & 4 deletions internal/config/flags.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,16 +11,16 @@ import "flag"

func (c *Config) handleFlagsConfig() {

flag.BoolVar(&c.showVersion, "v", defaultDisplayVersionAndExit, versionFlagHelp+" (shorthand)")
flag.BoolVar(&c.showVersion, "v", defaultDisplayVersionAndExit, versionFlagHelp+shorthandFlagSuffix)
flag.BoolVar(&c.showVersion, "version", defaultDisplayVersionAndExit, versionFlagHelp)

flag.IntVar(&c.minDigits, "md", defaultMinDigits, minDigitsFlagHelp+" (shorthand)")
flag.IntVar(&c.minDigits, "md", defaultMinDigits, minDigitsFlagHelp+shorthandFlagSuffix)
flag.IntVar(&c.minDigits, "min-digits", defaultMinDigits, minDigitsFlagHelp)

flag.IntVar(&c.minSpecialChars, "ms", defaultMinSpecialChars, minSpecialCharsFlagHelp+" (shorthand)")
flag.IntVar(&c.minSpecialChars, "ms", defaultMinSpecialChars, minSpecialCharsFlagHelp+shorthandFlagSuffix)
flag.IntVar(&c.minSpecialChars, "min-specials", defaultMinSpecialChars, minSpecialCharsFlagHelp)

flag.IntVar(&c.totalChars, "l", defaultTotalChars, totalCharsFlagHelp+" (shorthand)")
flag.IntVar(&c.totalChars, "l", defaultTotalChars, totalCharsFlagHelp+shorthandFlagSuffix)
flag.IntVar(&c.totalChars, "length", defaultTotalChars, totalCharsFlagHelp)

// Allow our function to override the default Help output
Expand Down

0 comments on commit cfc11c1

Please sign in to comment.