Repository navigation
docs(contributing): define limits for illustrative drafts before work acceptance - #4999
TechWizard9999 wants to merge 2 commits into
Conversation
… acceptance Signed-off-by: TechWizard9999 <119874392+TechWizard9999@users.noreply.github.com>
Signed-off-by: TechWizard9999 <119874392+TechWizard9999@users.noreply.github.com>
|
Thank you for your interest in contributing to OpenShell, @TechWizard9999. This project uses a vouch system for first-time contributors. Before submitting a pull request, you need to be vouched by a maintainer. To get vouched:
See CONTRIBUTING.md for details. |
|
All contributors have signed the DCO ✍️ ✅ |
|
I have read the DCO document and I hereby sign the DCO. |
|
Thanks for your look at this. We've found that too much documentation floods agent context, making work less efficient and sometimes prone to missing instructions. This is one of the cases I'd leave as is; the benefit is too low for the amount of docs. |
|
Hi @purp , Actually I am intrested in contributing to this repo do we have some easy - medium level issues if possibe can you point or assign to me , I would like to work on it |
Summary
Define limits, required context, and review expectations for illustrative drafts opened before work acceptance, and exempt draft pull requests from the issue acceptance check while in draft.
Related Issue
Closes #4952
Changes
### Illustrative Drafts for Unaccepted Worksubsection toCONTRIBUTING.mddefining purpose, scope limits, required context, review expectations, and positive/negative examples.CONTRIBUTING.mdline 32 to clarify that illustrative drafts never bypass the first-time contributor vouch requirement.CONTRIBUTING.mdline 449,AGENTS.md, and contributor skills (create-github-pr,review-github-pr,PULL_REQUEST_TEMPLATE.md) to permit illustrative drafts while in draft state, retaining the acceptance check when marked ready for review.tasks/scripts/check_issue_acceptance.cjsand add unit test coverage intasks/scripts/check_issue_acceptance.test.cjs.ready_for_reviewto.github/workflows/pr-issue-acceptance.ymlpull_request_target trigger types.Release Notes
Testing
tasks/scripts/check_issue_acceptance.test.cjs(22/22 tests passing)markdownlint-cli2(0 errors across 163 files).github/workflows/pr-issue-acceptance.ymlChecklist