Skip to content

docs: add issue approval readiness checklist - #174

Merged
El-swaggerito merged 1 commit into
Aegis-RWA:mainfrom
DevALVIN-24:issue-approval-readiness-checklist
Jul 29, 2026
Merged

docs: add issue approval readiness checklist#174
El-swaggerito merged 1 commit into
Aegis-RWA:mainfrom
DevALVIN-24:issue-approval-readiness-checklist

Conversation

@DevALVIN-24

Copy link
Copy Markdown
Contributor

Closes #140
Description:

Summary

This PR adds an issue approval readiness checklist to ensure contributors and reviewers have a clear set of expectations before considering an issue fully resolved and ready for evaluation.

Changes Included

  • Created Checklist: Added docs/issue-approval-readiness-checklist.md.
  • Linked in README: Updated the README.md under the "CI & Contributing" section to include a link to the new checklist as a mandatory read.

Acceptance Criteria Covered

  • Approval readiness checklist is added.
  • Checklist states that merged PRs are still subject to evaluation.
  • Testing expectations are included.
  • CI status expectations are included.
  • Acceptance criteria review is included.
  • README links to the checklist.

Motivation

Contributors previously assumed their work was complete simply because a PR was merged, which often left gaps in testing, documentation, or core acceptance criteria. This checklist acts as a definitive guide for evaluating implementation completeness, verifying tests and CI, assessing acceptance criteria, ensuring proper documentation, and noting any known limitations.


@El-swaggerito
El-swaggerito merged commit 08de4c8 into Aegis-RWA:main Jul 29, 2026
1 check passed
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.

Add Aegis Contracts issue approval readiness checklist

2 participants