-
Notifications
You must be signed in to change notification settings - Fork 1
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
6e29e4b
commit be4724f
Showing
1 changed file
with
109 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,109 @@ | ||
# Contributing guide | ||
|
||
Scanpy provides extensive [developer documentation][scanpy developer guide], most of which applies to this repo, too. | ||
This document will not reproduce the entire content from there. Instead, it aims at summarizing the most important | ||
information to get you started on contributing. | ||
|
||
We assume that you are already familiar with git and with making pull requests on GitHub. If not, please refer | ||
to the [scanpy developer guide][]. | ||
|
||
## Installing dev dependencies | ||
|
||
In addition to the packages needed to _use_ this package, you need additional python packages to _run tests_ and _build | ||
the documentation_. It's easy to install them using `pip`: | ||
|
||
```bash | ||
git clone https://github.com/MannLabs/scPortrait | ||
cd scPortrait | ||
pip install -e '.[dev]' | ||
``` | ||
|
||
## Code-style | ||
|
||
This project uses [pre-commit][] to enforce consistent code-styles. On every commit, pre-commit checks will either | ||
automatically fix issues with the code, or raise an error message. | ||
|
||
To enable pre-commit locally, simply run | ||
|
||
```bash | ||
pre-commit install | ||
``` | ||
|
||
in the root of the repository. Pre-commit will automatically download all dependencies when it is run for the first time. | ||
|
||
Alternatively, you can rely on the [pre-commit.ci][] service enabled on GitHub. If you didn't run `pre-commit` before | ||
pushing changes to GitHub it will automatically commit fixes to your pull request, or show an error message. | ||
|
||
If pre-commit.ci added a commit on a branch you still have been working on locally, simply use | ||
|
||
```bash | ||
git pull --rebase | ||
``` | ||
|
||
to integrate the changes into yours. | ||
While the [pre-commit.ci][] is useful, we strongly encourage installing and running pre-commit locally first to understand its usage. | ||
|
||
## Writing tests | ||
|
||
```{note} | ||
Remember to first install the package with pip install -e '.[dev]' | ||
``` | ||
TBD | ||
|
||
## Publishing a release | ||
|
||
### Updating the version number | ||
|
||
Before making a release, you need to update the version number. Please adhere to [Semantic Versioning][semver], in brief | ||
|
||
> Given a version number MAJOR.MINOR.PATCH, increment the: | ||
> | ||
> 1. MAJOR version when you make incompatible API changes, | ||
> 2. MINOR version when you add functionality in a backwards compatible manner, and | ||
> 3. PATCH version when you make backwards compatible bug fixes. | ||
> | ||
> Additional labels for pre-release and build metadata are available as extensions to the MAJOR.MINOR.PATCH format. | ||
Once you are done, run | ||
|
||
``` | ||
git push --tags | ||
``` | ||
|
||
to publish the created tag on GitHub. | ||
|
||
### Building and publishing the package on PyPI | ||
|
||
TBD | ||
|
||
## Writing documentation | ||
|
||
Please write documentation for new or changed features and use-cases. This project uses [sphinx][] with the following features: | ||
|
||
- Google-style docstrings | ||
- TBD | ||
|
||
See the [scanpy developer docs](https://scanpy.readthedocs.io/en/latest/dev/documentation.html) for more information | ||
on how to write documentation. | ||
|
||
### Tutorials with jupyter notebooks | ||
|
||
TBD | ||
|
||
#### Building the docs locally | ||
|
||
```bash | ||
cd docs | ||
# TBD | ||
``` | ||
|
||
<!-- Links --> | ||
|
||
[scanpy developer guide]: https://scanpy.readthedocs.io/en/latest/dev/index.html | ||
[cookiecutter-scverse-instance]: https://cookiecutter-scverse-instance.readthedocs.io/en/latest/template_usage.html | ||
[github quickstart guide]: https://docs.github.com/en/get-started/quickstart/create-a-repo?tool=webui | ||
[pre-commit.ci]: https://pre-commit.ci/ | ||
[pre-commit]: https://pre-commit.com/ | ||
[sphinx]: https://www.sphinx-doc.org/en/master/ | ||
[sphinx autodoc typehints]: https://github.com/tox-dev/sphinx-autodoc-typehints | ||
|