Skip to content

ask to share self-review notes#14185

Open
yiyixuxu wants to merge 7 commits into
mainfrom
self-review-share-notes
Open

ask to share self-review notes#14185
yiyixuxu wants to merge 7 commits into
mainfrom
self-review-share-notes

Conversation

@yiyixuxu

Copy link
Copy Markdown
Collaborator

What does this PR do?

Fixes # (issue)

Before submitting

yiyixuxu and others added 3 commits July 14, 2026 04:10
The PR template's self-review checkbox is unverifiable on its own — make the
notes a required, shareable artifact:

- self-review skill: new step to save each round's report and post all rounds
  on the PR (description or comment)
- contribution guide: sharing the notes is part of the AI-assisted
  contribution workflow; a checked box without notes counts as not done
- PR template: checkbox for having shared the notes
- .ai/AGENTS.md: mention the notes-sharing step in the self-review section

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@github-actions github-actions Bot added size/S PR with diff < 50 LOC documentation Improvements or additions to documentation CI labels Jul 14, 2026
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

…share

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment thread .ai/skills/self-review/SKILL.md Outdated
yiyixuxu and others added 3 commits July 13, 2026 21:48
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@yiyixuxu yiyixuxu requested review from DN6, sayakpaul and stevhliu July 14, 2026 08:08

@stevhliu stevhliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI documentation Improvements or additions to documentation size/S PR with diff < 50 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants