Skip to content

fix(workflows): give reviewers the complete PR diff - #1792

Merged
jamesadevine merged 1 commit into
mainfrom
jamesadevine/full-review-diffs
Aug 3, 2026
Merged

fix(workflows): give reviewers the complete PR diff#1792
jamesadevine merged 1 commit into
mainfrom
jamesadevine/full-review-diffs

Conversation

@jamesadevine

Copy link
Copy Markdown
Collaborator

Summary

  • remove the repository-owned 3,000-line truncation from both PR diff fetch paths
  • give all five reviewers the complete filtered diff while retaining generated-file exclusions
  • version the shared cache namespace so previously truncated entries cannot be restored

Validation

  • gh aw compile review-rust review-typescript review-tests review-security review-compiler-contract --strict --no-check-update
  • confirmed PR feat(safeoutputs): add GitHub issue outputs #1670 produces a 5,925-line filtered diff instead of stopping at 3,000

Remove the repository-owned 3,000-line truncation from both PR diff fetch paths and version the cache namespace so existing truncated entries cannot be restored.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: af5a1369-fd26-4781-a5b6-1bcd77e89a77
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@jamesadevine
jamesadevine merged commit d19e021 into main Aug 3, 2026
4 checks passed
@jamesadevine
jamesadevine deleted the jamesadevine/full-review-diffs branch August 3, 2026 13:18
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.

1 participant