smart-advisor / .github /pull_request_template.md
sajaahmed5
Smart Advisor deployment
74e8a7b
|
Raw
History Blame Contribute Delete
1.11 kB
# Pull Request
## What does this PR do?
<!-- A short, clear description of what changed and why -->
## Type of change
<!-- Check all that apply -->
- [ ] πŸ†• New feature
- [ ] πŸ› Bug fix
- [ ] πŸ“š Documentation update
- [ ] πŸ§ͺ Tests
- [ ] ♻️ Refactor (no behavior change)
- [ ] πŸ”§ Chore (deps, config, tooling)
## Which pillar does this touch?
<!-- Helps reviewers know what to focus on -->
- [ ] Knowledge Base (Roaa)
- [ ] RAG Pipeline (Fatema)
- [ ] Voice / STT / TTS (Saja)
- [ ] UI / Evaluation (Shahd)
- [ ] Cross-cutting / shared utils
## How did you test this?
<!-- Describe the tests you ran. Include screenshots/logs if relevant. -->
## Checklist before merging
- [ ] My code follows the project structure (right pillar folder)
- [ ] I tested my changes locally and they work
- [ ] I did not commit any API keys, .env files, or large model weights
- [ ] I added comments where the code is not self-explanatory
- [ ] I updated documentation if needed
- [ ] At least one teammate has reviewed this PR
## Notes for reviewers
<!-- Anything specific you want feedback on? -->