Skip to content

Fix PR context renderer test assertion - #52201

Merged
pelikhan merged 2 commits into
mainfrom
copilot/gh-aw-fix-with-copilot
Aug 12, 2026
Merged

Fix PR context renderer test assertion#52201
pelikhan merged 2 commits into
mainfrom
copilot/gh-aw-fix-with-copilot

Conversation

Copilot AI commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

The PR checkout integration test expected the obsolete shell-based prompt inclusion and failed after prompt files moved to the structured renderer configuration.

  • Update PR context detection
    • Assert the renderer’s file entry instead of a removed cat command.
    • Preserve positive and negative coverage across comment and non-comment triggers.
{"file":"pr_context_prompt.md"}

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issues with GitHub Actions job Fix PR context renderer test assertion Aug 12, 2026
Copilot AI requested a review from pelikhan August 12, 2026 02:06
@pelikhan
pelikhan marked this pull request as ready for review August 12, 2026 02:06
Copilot AI balanced review requested due to automatic review settings August 12, 2026 02:06
@pelikhan
pelikhan merged commit f791e7b into main Aug 12, 2026
@pelikhan
pelikhan deleted the copilot/gh-aw-fix-with-copilot branch August 12, 2026 02:06

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the PR checkout integration test to match the structured prompt renderer configuration.

Changes:

  • Replaces obsolete shell-command assertions with renderer file-entry assertions.
  • Updates comments to describe renderer-based prompt inclusion.
Show a summary per file
File Description
pkg/workflow/pr_checkout_test.go Validates pr_context_prompt.md in renderer configuration.

Review details

Tip

Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

@albertoblue87-netizen albertoblue87-netizen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listo

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.

4 participants