Skip to content

Commit

Permalink
chore: pr feedback
Browse files Browse the repository at this point in the history
Co-authored-by: Cédric Fabianski <[email protected]>
  • Loading branch information
gotbadger and cfabianski authored Oct 4, 2023
1 parent 9bedf73 commit 9c2fafe
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/commands/process/settings/settings.go
Original file line number Diff line number Diff line change
Expand Up @@ -364,7 +364,7 @@ func FromOptions(opts flag.Options, versionMeta *version_check.VersionMeta) (Con
}

if config.Scan.DiffBaseBranch != "" {
if config.Report.Report != flag.ReportSecurity && config.Report.Report != flag.ReportSaaS {
if !slices.Contains([]string{flag.ReportSecurity, flag.ReportSaaS}, config.Report.Report) {

Check failure on line 367 in internal/commands/process/settings/settings.go

View workflow job for this annotation

GitHub Actions / build

undefined: slices

Check failure on line 367 in internal/commands/process/settings/settings.go

View workflow job for this annotation

GitHub Actions / validate

undefined: slices

Check failure on line 367 in internal/commands/process/settings/settings.go

View workflow job for this annotation

GitHub Actions / build

undefined: slices
return Config{}, errors.New("diff base branch is only supported for the security report")
}
}
Expand Down

0 comments on commit 9c2fafe

Please sign in to comment.