Skip to content

Remove verbose pipeline template render logs#381

Merged
marccampbell merged 1 commit into
mainfrom
remove-pipeline-template-render-logs
Jun 8, 2026
Merged

Remove verbose pipeline template render logs#381
marccampbell merged 1 commit into
mainfrom
remove-pipeline-template-render-logs

Conversation

@marccampbell

Copy link
Copy Markdown
Contributor

Summary

  • Remove verbose pipeline raw template, template data, and rendered template logs.
  • Keep warning/error logs for template parse, execute, and issue lookup failures.

Verification

  • go test ./pkg/hub -run 'TestInjectTemplateDataMergesOutputs|TestPipelineEntryInjectIncludesInitialPlanInstruction|TestWorkflowPipelineContextUsesWorkspaceIssueTracker'
  • go test ./...

@greptile-apps

greptile-apps Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Comments Outside Diff (1)

  1. pkg/hub/pipeline_runner.go, line 1055 (link)

    P2 Inconsistent log removal between Linear and GitHub paths

    The equivalent "fetched issue" log for Linear ([pipeline] fetched issue %s: identifier=%s title=%s) was removed in this PR, but this GitHub counterpart on line 1055 was left in. If the goal is to strip all verbose fetch/template detail logs symmetrically, this line should be removed too.

    Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Reviews (1): Last reviewed commit: "Remove pipeline template render logs" | Re-trigger Greptile

@marccampbell marccampbell merged commit 27b240e into main Jun 8, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants