Skip to content

ci: dogfood — pin diffctx's review context to every pull request - #260

Merged
nikolay-e merged 2 commits into
mainfrom
feature/pr-review-context
Sep 3, 2026
Merged

ci: dogfood — pin diffctx's review context to every pull request#260
nikolay-e merged 2 commits into
mainfrom
feature/pr-review-context

Conversation

@nikolay-e

Copy link
Copy Markdown
Owner

Every PR runs the published composite action on its own base..head and pins the result as one sticky comment plus a downloadable artifact, so reviewers (and /qa) read diffctx's selection instead of the raw diff. QA.md gains the matching checklist item; the action docs describe the job. This PR is the first to receive the comment — the proof it works.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

diffctx review context

Range ad571a3b420077229c39d219166b6ae61b4c65c7..7138493b4f402f803a11e9b156a9dc5d955b9f2d7455 tokens (o200k_base), 13 fragments across 2 changed files. Empty: false.

Review this instead of the raw diff: download the context (artifact diffctx-review-context, 14 days).

1 changed file(s) withheld by exclusion policy (.diffctx/ignore or secret paths). — read those in the raw diff.

Changed files as diffctx lists them (omitted = no fragment fit)
  • QA.md
  • docs/product/github-action.md

@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

@nikolay-e
nikolay-e merged commit 25cc029 into main Sep 3, 2026
25 checks passed
@nikolay-e
nikolay-e deleted the feature/pr-review-context branch September 3, 2026 11:26
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