Skip to content

docs: document Sashiko-for-Sashiko local and PR review workflow - #592

Merged
rgushchin merged 1 commit into
mainfrom
docs/sashiko-for-sashiko-workflow
Sep 29, 2026
Merged

rgushchin merged 1 commit into
mainfrom
docs/sashiko-for-sashiko-workflow

Conversation

@rgushchin

Copy link
Copy Markdown
Member

Updates README.md and GEMINI.md to describe the two-stage Sashiko-for-Sashiko development model:

  1. Local Agentic Loop: Developers and coding agents run sashiko review --project sashiko on proposed commits after passing make check-pr, resolving any findings locally before opening or updating a pull request.
  2. Upstream GitHub PR Review: Once a pull request is opened or updated, the upstream Sashiko instance (sashiko.sashiko.dev) posts automated review comments (sashiko-bot) on GitHub. Pull request authors are required to act on every finding by either fixing valid defects and pushing an update, or replying with concrete code evidence explaining why a finding is a false positive and proposing corresponding prompt or workflow improvements.

Update README.md and GEMINI.md to describe the two-stage self-review
development model for the Sashiko repository:
1. Local Agentic Loop: Developers and coding agents run sashiko review
   with the sashiko project profile on proposed commits after passing
   make check-pr, resolving any findings locally before opening or
   updating a pull request.
2. Upstream GitHub PR Review: Once a pull request is opened or updated,
   the upstream Sashiko instance posts automated review comments on
   GitHub. Pull request authors are required to act on every finding by
   either fixing valid defects and pushing an update, or replying with
   concrete code evidence explaining why a finding is a false positive
   and proposing corresponding prompt or workflow improvements.

Signed-off-by: Roman Gushchin <roman.gushchin@linux.dev>
@sashiko-bot

sashiko-bot Bot commented Sep 29, 2026

Copy link
Copy Markdown

Sashiko review

✓ No issues found across 1 commit.

Full review log on sashiko.sashiko.dev

@rgushchin
rgushchin merged commit de7e866 into main Sep 29, 2026
3 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.

1 participant