Skip to content

Catalog dataset guidelines and add OpenHands PR review#186

Open
neubig wants to merge 2 commits intomainfrom
openhands/catalog-dataset-guidelines
Open

Catalog dataset guidelines and add OpenHands PR review#186
neubig wants to merge 2 commits intomainfrom
openhands/catalog-dataset-guidelines

Conversation

@neubig
Copy link
Copy Markdown
Contributor

@neubig neubig commented May 2, 2026

Summary

  • Expand AGENTS.md with strict dataset incorporation guidelines, including generator-first sample creation, schema/API/SFT validation expectations, common prior-PR failure modes, and a required design-decision catalog for unclear choices.
  • Add the OpenHands PR review workflow from OpenHands/extensions so PRs can be reviewed automatically or on demand.
  • Add repository-specific /codereview guidance that instructs the review agent to read AGENTS.md and verify the dataset guidelines and PR-description requirements.

Testing

  • git --no-pager diff --check
  • Python marker/trailing-newline validation for AGENTS.md, .agents/skills/custom-codereview-guide.md, and .github/workflows/pr-review-by-openhands.yml
  • No application-code tests were added or run because this change only updates documentation and GitHub/workflow configuration.

Fixes #185

This PR was created by an AI agent (OpenHands) on behalf of the user.

@neubig can click here to continue refining the PR

Co-authored-by: openhands <openhands@all-hands.dev>
@neubig neubig marked this pull request as ready for review May 2, 2026 00:18
@neubig neubig enabled auto-merge (squash) May 2, 2026 00:18
@yueqis yueqis self-requested a review May 2, 2026 00:20
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.

Catalog guidelines in AGENTS.md and PR review

3 participants