Skip to content

Commit 53c5e8a

Browse files
leodidoona-agent
andcommitted
fix: resolve linting issues
- Check error return value from MarkFlagRequired using blank identifier - Remove unused getEnvOrDefault function Co-authored-by: Ona <no-reply@ona.com>
1 parent 3e05a1f commit 53c5e8a

2 files changed

Lines changed: 1 addition & 9 deletions

File tree

cmd/sign-cache.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ func init() {
4949
plumbingCmd.AddCommand(signCacheCmd)
5050
signCacheCmd.Flags().String("from-manifest", "", "Path to newline-separated artifact paths file")
5151
signCacheCmd.Flags().Bool("dry-run", false, "Log actions without signing or uploading")
52-
signCacheCmd.MarkFlagRequired("from-manifest")
52+
_ = signCacheCmd.MarkFlagRequired("from-manifest")
5353
}
5454

5555
// runSignCache implements the main signing logic

pkg/leeway/signing/attestation.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -355,12 +355,4 @@ func validateSigstoreEnvironment() error {
355355

356356
log.Debug("Sigstore environment validation passed")
357357
return nil
358-
}
359-
360-
// getEnvOrDefault returns environment variable value or default
361-
func getEnvOrDefault(key, defaultValue string) string {
362-
if value := os.Getenv(key); value != "" {
363-
return value
364-
}
365-
return defaultValue
366358
}

0 commit comments

Comments
 (0)