feat: use a pointer for the deprecated option

Author: ldez

pkg/config/loader.go

@@ -336,11 +336,10 @@ func (l *Loader) handleDeprecation() error {
 	}
 	l.cfg.Output.ShowStats = l.cfg.Run.ShowStats || l.cfg.Output.ShowStats
 
-	// The 2 options are true by default.
 	// Deprecated since v1.63.0
-	if !l.cfg.Output.UniqByLine {
+	if l.cfg.Output.UniqByLine != nil {
 		l.log.Warnf("The configuration option `output.uniq-by-line` is deprecated, please use `issues.uniq-by-line`")
-		l.cfg.Issues.UniqByLine = l.cfg.Output.UniqByLine
+		l.cfg.Issues.UniqByLine = *l.cfg.Output.UniqByLine
 	}
 
 	// Deprecated since v1.57.0

pkg/config/output.go

@@ -52,7 +52,7 @@ type Output struct {
 	Format string `mapstructure:"format"`
 
 	// Deprecated: use [Issues.UniqByLine] instead.
-	UniqByLine bool `mapstructure:"uniq-by-line"`
+	UniqByLine *bool `mapstructure:"uniq-by-line"`
 }
 
 func (o *Output) Validate() error {