Fix missing lint errors

Author: edif2008

internals/cli/clip/fakeclip/clip.go

@@ -1,4 +1,4 @@
-// +build !production
+//go:build !production
 
 // Package fakeclip provides fake implementations of the clip.Clipper interface
 // to be used for testing.

internals/cli/cobra_template.go

@@ -378,21 +378,21 @@ func numFlags(flagSet pflag.FlagSet) int {
 
 // UsageTemplate is the custom usage template of the command.
 // Changes in comparison to cobra's default template:
-// 1. Usage section
-//	  a. `[flags]` is placed before the arguments.
-//	  b. All arguments are put in the usage in their proper order.
-//    c. Where applicable, the argument name is replaced with its placeholder.
-// 2. Commands section
-// 	  a. For the root command (`secrethub`) the commands are grouped into
-//		 `Management commands` and `Commands`
-// 2. Flags section
-// 	  a. Flag's type was removed.
-// 	  b. The help text for flags is well divided into its own column, thus
-//		 making the visibility of the flags better.
-// 	  c. At the end of a flag's help text, the name of its environment variable is
-//       displayed between brackets.
-//    d. The section is hidden if the only flag is `--help`.
-// 4. Arguments section (created by us)
+//  1. Usage section
+//     a. `[flags]` is placed before the arguments.
+//     b. All arguments are put in the usage in their proper order.
+//     c. Where applicable, the argument name is replaced with its placeholder.
+//  2. Commands section
+//     a. For the root command (`secrethub`) the commands are grouped into
+//     `Management commands` and `Commands`
+//  2. Flags section
+//     a. Flag's type was removed.
+//     b. The help text for flags is well divided into its own column, thus
+//     making the visibility of the flags better.
+//     c. At the end of a flag's help text, the name of its environment variable is
+//     displayed between brackets.
+//     d. The section is hidden if the only flag is `--help`.
+//  4. Arguments section (created by us)
 var UsageTemplate = `Usage:
 {{if .Cmd.Runnable}} {{(useLine .Cmd .Args)}}{{end}}
 {{- if .Cmd.HasAvailableSubCommands}}  {{ .Cmd.CommandPath}} [command]{{end}}

internals/cli/mlock/mlock_unavailable.go

@@ -1,4 +1,4 @@
-// +build android darwin nacl netbsd plan9 windows
+//go:build android || darwin || nacl || netbsd || plan9 || windows
 
 package mlock
 

internals/secrethub/tpl/v2.go

@@ -84,17 +84,17 @@ type parserV2 struct{}
 // Parse parses a secret template from a raw string.
 //
 // Syntax rules:
-// - A secret template can contain references to secrets in secret tags. A
-//   secret tag is enclosed in double brackets: `{{ path/to/secret }}`.
-// - A secret template can contain references to variables in variable tags. A
-//   variable tag is enclosed between ${ and }: `${ variable }`.
-// - Extra spaces can be added just after the opening delimiter and just before the
-//   closing delimiter of a tag: {{ path/to/secret }} has the same output as
-//   {{path/to/secret}} has.
-// - Secret tags can also contain variable tags: `{{ path/with/${var}/to/secret }}`
-// - Variable tags cannot contain secret tags.
-// - Secret tags cannot contain secret tags (they cannot be nested).
-// - Variable tags cannot contain variable tags (they cannot be nested).
+//   - A secret template can contain references to secrets in secret tags. A
+//     secret tag is enclosed in double brackets: `{{ path/to/secret }}`.
+//   - A secret template can contain references to variables in variable tags. A
+//     variable tag is enclosed between ${ and }: `${ variable }`.
+//   - Extra spaces can be added just after the opening delimiter and just before the
+//     closing delimiter of a tag: {{ path/to/secret }} has the same output as
+//     {{path/to/secret}} has.
+//   - Secret tags can also contain variable tags: `{{ path/with/${var}/to/secret }}`
+//   - Variable tags cannot contain secret tags.
+//   - Secret tags cannot contain secret tags (they cannot be nested).
+//   - Variable tags cannot contain variable tags (they cannot be nested).
 func (p parserV2) Parse(raw string, line, column int) (Template, error) {
 	parser := newV2Parser(bytes.NewBufferString(raw), line, column)
 

internals/winrm/client.go

@@ -46,7 +46,7 @@ type Config struct {
 	CaCert         []byte
 }
 
-/// authMethod gives an interface to retrieve the valid credentials or defaults.
+// authMethod gives an interface to retrieve the valid credentials or defaults.
 // The Client always needs both values for certificates authentication and basic authentication.
 // So we use implementations of this interface to give us the values without having to know the implementation.
 // The necessity of this is because of the weird interface giving by our dependency on the masterzen/winrm library.

internals/winrm/copy.go

@@ -22,9 +22,9 @@ var (
 
 // doCopy copies the contents from the io.Reader into the to path.
 // The copy process is done in multiple steps:
-//     1. The content is uploaded to a temporary file in chunks
-//     2. The content is restored from the chunks into a single file at the target location.
-//     3. The temporary file is removed.
+//  1. The content is uploaded to a temporary file in chunks
+//  2. The content is restored from the chunks into a single file at the target location.
+//  3. The temporary file is removed.
 //
 // Progress is reported in the progress channel given as a variable.
 func doCopy(client *winrm.Client, in io.Reader, toPath string, progress chan int) error {
@@ -75,7 +75,7 @@ func uploadContent(client *winrm.Client, maxChunks int, filePath string, reader
 }
 
 // uploadChunks uploads the content by dividing the content into multiple chunks.
-// The chunks are combined into a single file.
+//  The chunks are combined into a single file.
 // The chunks are used to get around the maximum command line size limit.
 // This allows us to use the winRM connection for uploading files.
 func uploadChunks(client *winrm.Client, filePath string, maxChunks int, reader io.Reader) (done bool, err error) {