Skip to content

chore(ci): implement PR template & review checklist (#167)#186

Merged
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
SamixYasuke:chore/implement-pr-template-
Feb 25, 2026
Merged

chore(ci): implement PR template & review checklist (#167)#186
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
SamixYasuke:chore/implement-pr-template-

Conversation

@SamixYasuke

Copy link
Copy Markdown
Contributor

Implement PR Template & Review Checklist

Closes #167

What does this PR do?

  • PR template: Rewritten to require linked issue, NestJS-focused error handling/DTO/guards/logging checks, Swagger confirmation, and explicit test evidence.
  • CONTRIBUTING: Updated to make the PR template mandatory and treat missing checklist items/test evidence as merge blockers; clarified reviewer error-handling checks.
    Tooling:
  • Added commitlint.config.cjs extending @commitlint/config-conventional so Conventional Commit messages pass Husky hooks.

@SamixYasuke

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER please review and merge. Thanks

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Please fix CI

@SamixYasuke

Copy link
Copy Markdown
Contributor Author

On it

@SamixYasuke

Copy link
Copy Markdown
Contributor Author

@RUKAYAT-CODER Please review

@RUKAYAT-CODER RUKAYAT-CODER merged commit 5080a87 into rinafcode:main Feb 25, 2026
6 of 8 checks 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.

Implement PR Template & Review Checklist

2 participants