Skip to content

Commit

Permalink
Merge pull request #1691 from imjasonh/no-slog
Browse files Browse the repository at this point in the history
drop spammy slog attributes
  • Loading branch information
imjasonh authored Dec 11, 2024
2 parents 374e31e + e721c44 commit 5318602
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions pkg/build/pipeline.go
Original file line number Diff line number Diff line change
Expand Up @@ -237,17 +237,6 @@ func (r *pipelineRunner) runPipeline(ctx context.Context, pipeline *config.Pipel
log.Infof("running step %q", id)
}

slogs := []any{}
if pipeline.Name != "" {
slogs = append(slogs, "name", pipeline.Name)
}
if pipeline.Uses != "" {
slogs = append(slogs, "uses", pipeline.Uses)
}
if len(slogs) != 0 {
ctx = clog.WithLogger(ctx, log.With(slogs...))
}

command := buildEvalRunCommand(pipeline, debugOption, workdir, pipeline.Runs)
if err := r.runner.Run(ctx, r.config, envOverride, command...); err != nil {
if err := r.maybeDebug(ctx, pipeline.Runs, envOverride, command, workdir, err); err != nil {
Expand Down

0 comments on commit 5318602

Please sign in to comment.