Skip to content

feat: Add PR template from main OpenHands repository#653

Closed
jamiechicago312 wants to merge 2 commits into
mainfrom
openhands/add-pr-template
Closed

feat: Add PR template from main OpenHands repository#653
jamiechicago312 wants to merge 2 commits into
mainfrom
openhands/add-pr-template

Conversation

@jamiechicago312

@jamiechicago312 jamiechicago312 commented Apr 6, 2026

Copy link
Copy Markdown
Member
  • A human has tested these changes.

Why

To standardize pull request descriptions and ensure consistent information is provided by all contributors. This aligns the OpenHands-CLI repository with the main OpenHands repository's PR workflow.

Summary

  • Copied PR template from OpenHands/OpenHands repository
  • Added standard sections: Why, Summary, Issue Number, How to Test, Video/Screenshots, Type, Notes
  • Included checkbox for human testing verification

Issue Number

Fixes #652

How to Test

  1. Create a new PR in this repository
  2. Verify that the PR template automatically appears in the description field
  3. Confirm all sections from the template are present

Video/Screenshots

N/A - This is a documentation/configuration change

Type

  • Bug fix
  • Feature
  • Refactor
  • Breaking change
  • Docs / chore

Notes

This template will be automatically applied to all new pull requests created in the repository, helping maintain consistency across contributions.


🚀 Try this PR

uvx --python 3.12 git+https://github.com/OpenHands/OpenHands-CLI.git@openhands/add-pr-template

- Copy PR template from OpenHands/OpenHands repository
- Add standard sections: Why, Summary, Issue Number, How to Test, Video/Screenshots, Type, Notes
- Include checkbox for human testing verification

Co-authored-by: openhands <openhands@all-hands.dev>
@openhands-ai openhands-ai Bot mentioned this pull request Apr 6, 2026
@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL663290686% 
report-only-changed-files is enabled. No files were changed during this commit :)

@jamiechicago312 jamiechicago312 marked this pull request as ready for review April 6, 2026 22:14

@all-hands-bot all-hands-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🟢 Good taste - Clean template addition.

Verdict: ✅ Worth merging. This is a straightforward standardization effort.

Key Insight: It's a markdown template, not rocket science. The structure is sensible and aligns with the main repo's workflow. Ship it.

Comment thread .github/pull_request_template.md Outdated
Comment thread .github/pull_request_template.md Outdated

@enyst enyst 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.

In reality, I think we figured out in the sdk that maybe the human testing isn't all that human anymore. That is, purely manual testing on one's local machine is not really the need nor the practice for a while, with some exceptions.

Let's see where we land there and pick that up, maybe? Both the SDK and CLI are small enough codebases that already moved a bit past manual verification, I think.

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open for 40 days with no activity. Remove the stale label or leave a comment, otherwise it will be closed in 10 days.

@github-actions github-actions Bot added the Stale label May 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

This PR was closed because it had no activity for 50 days. If you feel this was closed in error, and you would like to continue the PR, please resubmit or let us know.

@github-actions github-actions Bot closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] New Pr Template

4 participants