Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pull request template #111

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
<!-- Thank you for your contribution to DeepSensor! -->
<!-- Please fill out the details in the pull request description below as appropriate -->
<!-- Ensure that you abide by the code of conduct -->

## :pencil: Description
<!-- Please provide a clear description of the changes that are introduced in this pull request and why -->
<!-- It also helps to explain any justification or options you're aware of -->
<!-- If this pull request relates to an issue, please include the issue number here, e.g. #42 -->


## :white_check_mark: Checklist before requesting a review
(See the contributing guide for more details on these steps.)
- [ ] I have installed developer dependencies with `pip install -r requirements/requirements.dev.txt` and running `pre-commit install` (or alternatively, manually running `ruff format` before commiting)

If changing or adding source code:
- [ ] tests are included and are passing (run `pytest`).
- [ ] documentation is included or updated as relevant, including docstrings.

If changing or adding documentation:
- [ ] docs build successfully (`jupyter-book build docs --all`) and the changes look good from a manual inspection of the HTML in `docs/_build/html/`.
Loading