|
| 1 | +# Pull Request |
| 2 | + |
| 3 | +> When to use: Use this default template for most changes (features, small fixes, refactors, docs). If your change is a bug fix, performance optimization, maintenance/chore, or a breaking change, consider using one of the specialized templates under `.github/PULL_REQUEST_TEMPLATE/`. |
| 4 | +
|
| 5 | +Thank you for contributing to OntoGraph! Please fill out the sections below to help us review efficiently. |
| 6 | + |
| 7 | +## Summary |
| 8 | + |
| 9 | +- Link to issue(s): Closes #<issue-number> (or) Related to #<issue-number> |
| 10 | +- Brief description of the change: |
| 11 | + |
| 12 | +## Type of change |
| 13 | + |
| 14 | +Select one or more: |
| 15 | +- [ ] Bug fix (non-breaking change that fixes an issue) |
| 16 | +- [ ] Feature (non-breaking change that adds functionality) |
| 17 | +- [ ] Documentation update |
| 18 | +- [ ] Refactor/maintenance (no functional change) |
| 19 | +- [ ] Performance improvement |
| 20 | +- [ ] CI/CD or tooling |
| 21 | +- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) |
| 22 | + |
| 23 | +## Changes |
| 24 | + |
| 25 | +- High-level list of changes (modules, major functions/classes touched) |
| 26 | +- Any data files or schema changes |
| 27 | +- Any API changes (inputs/outputs, public interfaces) |
| 28 | + |
| 29 | +## Screenshots or demos (optional) |
| 30 | + |
| 31 | +If relevant, include images, terminal outputs, or notebook snippets. |
| 32 | + |
| 33 | +## Checklist |
| 34 | + |
| 35 | +- Code quality & style |
| 36 | + - [ ] Follows project style and conventions |
| 37 | + - [ ] Pre-commit hooks run locally |
| 38 | + |
| 39 | + ```bash |
| 40 | + pre-commit run -a |
| 41 | + ``` |
| 42 | +- Tests |
| 43 | + - [ ] Unit tests added/updated |
| 44 | + - [ ] All tests pass locally (`pytest`) |
| 45 | + - [ ] Includes edge cases (empty/null, invalid input, large inputs) |
| 46 | +- Documentation |
| 47 | + - [ ] User docs updated (MkDocs in `docs/` as needed) |
| 48 | + - [ ] README updated if behavior or usage changed |
| 49 | + - [ ] Docstrings and comments added where helpful |
| 50 | +- Reliability |
| 51 | + - [ ] No secrets or credentials in code or history |
| 52 | + - [ ] Backwards compatibility considered; migration notes added if breaking |
| 53 | + - [ ] Performance implications considered (time/memory), benchmarks if relevant |
| 54 | +- Project hygiene |
| 55 | + - [ ] Linked issues referenced with `Closes #...` or `Related to #...` |
| 56 | + - [ ] Changelog entry prepared (see `CHANGELOG.md`) |
| 57 | + - [ ] Labels applied (bug, enhancement, docs, etc.) |
| 58 | +- Community standards |
| 59 | + - [ ] I have read and agree to the [Code of Conduct](../CODE_OF_CONDUCT.md) |
| 60 | + |
| 61 | +## Test plan |
| 62 | + |
| 63 | +Describe how you tested the changes. Include commands, datasets, and expected outcomes. If applicable, reference tests in `tests/`: |
| 64 | + |
| 65 | +```bash |
| 66 | +# Example (adjust as needed) |
| 67 | +pytest -q |
| 68 | +``` |
| 69 | + |
| 70 | +## Breaking changes (if any) |
| 71 | + |
| 72 | +- Describe what breaks and how users should migrate. |
| 73 | +- Provide a migration example or script if possible. |
| 74 | + |
| 75 | +## Additional context |
| 76 | + |
| 77 | +- Risks, trade-offs, or follow-up tasks |
| 78 | +- Dependencies or environment changes |
| 79 | +- Related PRs |
0 commit comments