Skip to content

Commit b2bdfa4

Browse files
committed
chore: add PR template and CODEOWNERS
1 parent 7524888 commit b2bdfa4

2 files changed

Lines changed: 52 additions & 0 deletions

File tree

.github/CODEOWNERS

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
* @EmeditWeb
2+
src/ @EmeditWeb
3+
test/ @EmeditWeb

.github/pull_request_template.md

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
## Summary
2+
3+
Closes #[issue number]
4+
5+
Briefly describe what this PR does in 2-3 sentences.
6+
7+
## This repo is for the NestJS backend API only
8+
9+
Before submitting, confirm your changes belong here:
10+
11+
- [ ] My changes are inside src/ or test/
12+
- [ ] I have NOT added React, React Native,
13+
or frontend component files
14+
- [ ] I have NOT added Rust or Soroban contract code
15+
- [ ] This is NestJS/TypeScript backend work
16+
17+
## Type of change
18+
19+
- [ ] Bug fix
20+
- [ ] New endpoint
21+
- [ ] New service or module
22+
- [ ] Database migration
23+
- [ ] Background job
24+
- [ ] Test coverage
25+
26+
## Testing
27+
28+
- [ ] npm run build passes with zero TypeScript errors
29+
- [ ] npm test passes — all 184+ existing tests pass
30+
- [ ] No new `any` types introduced anywhere
31+
- [ ] Swagger decorators added to every new endpoint
32+
- [ ] Migration file created for any schema changes
33+
- [ ] New unit tests written for new service methods
34+
35+
## Context files reviewed
36+
37+
- [ ] context/architecture-context.md
38+
- [ ] context/code-standards.md
39+
- [ ] context/progress-tracker.md updated
40+
41+
## Mandatory before requesting review
42+
43+
Running these must all exit 0:
44+
npm run build
45+
npm test
46+
47+
If either fails, fix it before opening this PR.
48+
PRs with failing CI checks will be closed without review.
49+
PRs that reduce the test count will be rejected.

0 commit comments

Comments
 (0)