feat: Add PR template from main OpenHands repository#653
feat: Add PR template from main OpenHands repository#653jamiechicago312 wants to merge 2 commits into
Conversation
- 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>
Coverage Report •
|
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
🟢 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.
enyst
left a comment
enyst
left a comment
There was a problem hiding this comment.
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.
|
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. |
4 participants
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
Issue Number
Fixes #652
How to Test
Video/Screenshots
N/A - This is a documentation/configuration change
Type
Notes
This template will be automatically applied to all new pull requests created in the repository, helping maintain consistency across contributions.
🚀 Try this PR