Development guide¶
This page provides procedures and guidelines for developing and contributing to Kafkit.
Scope of contributions¶
Kafkit is an open source package, meaning that you can contribute to Kafkit itself, or fork Kafkit for your own purposes.
Since Kafkit is intended for internal use by Rubin Observatory, community contributions can only be accepted if they align with Rubin Observatory’s aims. For that reason, it’s a good idea to propose changes with a new GitHub issue before investing time in making a pull request.
Kafkit is developed by the LSST SQuaRE team.
Setting up a local development environment¶
To develop Kafkit, create a virtual environment with your method of choice (like virtualenvwrapper) and then clone or fork, and install:
git clone https://github.com/lsst-sqre/kafkit.git
cd kafkit
make init
This init step does three things:
- Installs Kafkit in an editable mode with its “dev” extra that includes test and documentation dependencies.
- Installs pre-commit and tox.
- Installs the pre-commit hooks.
Pre-commit hooks¶
The pre-commit hooks, which are automatically installed by running the make init command on set up, ensure that files are valid and properly formatted. Some pre-commit hooks automatically reformat code:
seed-isort-config
- Adds configuration for isort to the
pyproject.toml
file. isort
- Automatically sorts imports in Python modules.
black
- Automatically formats Python code.
blacken-docs
- Automatically formats Python code in reStructuredText documentation and docstrings.
When these hooks fail, your Git commit will be aborted. To proceed, stage the new modifications and proceed with your Git commit.
Running tests¶
One way to test the library is by running pytest from the root of the source repository:
pytest
You can also run tox, which tests the library the same way that the CI workflow does:
tox
To see a listing of test environments, run:
tox -av
Building documentation¶
Documentation is built with Sphinx:
tox -e docs
The build documentation is located in the docs/_build/html
directory.
Updating the change log¶
Each pull request should update the change log (CHANGELOG.rst
).
Add a description of new features and fixes as list items under a section at the top of the change log called “Unreleased:”
Unreleased
----------
- Description of the feature or fix.
If the next version is known (because Kafkit’s master branch is being prepared for a new major or minor version), the section may contain that version information:
X.Y.0 (unreleased)
------------------
- Description of the feature or fix.
If the exact version and release date is known (because a release is being prepared), the section header is formatted as:
X.Y.0 (YYYY-MM-DD)
------------------
- Description of the feature or fix.
Style guide¶
Code¶
Documentation¶
- Follow the LSST DM User Documentation Style Guide, which is primarily based on the Google Developer Style Guide.
- Document the Python API with numpydoc-formatted docstrings. See the LSST DM Docstring Style Guide.
- Follow the LSST DM ReStructuredTextStyle Guide. In particular, ensure that prose is written one-sentence-per-line for better Git diffs.