# Code Review Workflow {#code-review-workflow}


Standards and process for code reviews.


## Pull Request Process {#pull-request-process}






```mermaid
graph LR
    Branch[Feature Branch] --> PR[Open PR]
    PR --> CI[CI Checks Pass]
    CI --> Review[Code Review]
    Review -->|Approved| Merge[Merge to Develop]
    Review -->|Changes| Fix[Address Feedback]
    Fix --> CI
```






## Before Opening a PR {#before-opening-a-pr}


1. Rebase on latest `develop`
2. Ensure all tests pass locally
3. Run linting: `yarn lint`
4. Self-review your diff
5. Write a clear PR description


## PR Template {#pr-template}


```markdown
## What Changed

Brief description of changes.

## Why

Business context or issue reference.

## How to Test

1. Step-by-step testing instructions
2. Expected outcomes

## Screenshots

Before/after if UI changes.

## Checklist

- [ ] Tests added/updated
- [ ] Documentation updated
- [ ] No console.log statements
- [ ] Types properly defined
```


## Reviewer Checklist {#reviewer-checklist}


| Area           | Check                          |
| -------------- | ------------------------------ |
| Logic          | Correct business logic         |
| Security       | No vulnerabilities introduced  |
| Performance    | No N+1 queries, proper indexes |
| Types          | Proper TypeScript types        |
| Tests          | Adequate test coverage         |
| Naming         | Clear variable/function names  |
| Error Handling | Proper exception handling      |
| Tenant Scoping | `tenantId` properly applied    |


## Review Response Times {#review-response-times}


| Priority | Target Response |
| -------- | --------------- |
| Critical | < 2 hours       |
| Normal   | < 24 hours      |
| Low      | < 48 hours      |


## Related Pages {#related-pages}


- [Git Workflow](https://sourcey-gauzy.pages.dev/development/git-workflow/index.md) — branching model
- [CI Test Pipeline](https://sourcey-gauzy.pages.dev/testing/ci-test-pipeline/index.md) — CI checks
- [Development Guide](https://sourcey-gauzy.pages.dev/development/development-guide/index.md) — setup