Development guide¶
This page provides procedures and guidelines for developing and contributing to Ook.
Scope of contributions¶
Ook is an open source package, meaning that you can contribute to Ook itself, or fork Squarebot for your own purposes.
Since Ook 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.
Ook is developed by the Rubin Observatory SQuaRE team.
Setting up a local development environment¶
Local development requires:
Python 3.13
Docker or a compatible container runtime
First, clone the Ook repository:
git clone https://github.com/lsst-sqre/ook.git
cd ook
Ook is a Python project that should be developed within a virtual environment. Create this virtual environment with UV:
uv venv
source .venv/bin/activate
Then install the development dependencies and pre-commit hooks:
make init
Pre-commit hooks¶
The pre-commit hooks ensure that files are valid and properly formatted. The hooks are automatically installed by running the make init.
When these hooks fail, your Git commit will be aborted. To proceed, stage the new modifications and proceed with your Git commit.
Refer to the .pre-commit-config.yaml
file for the list of hooks that are run.
Running tests¶
To test all components of Ook, run nox, which tests the library the same way that the GitHub Actions CI workflow does.
nox
uv run --group=nox nox
File linting, type checking, and unit tests are run as separate nox sessions:
nox -s lint
nox -s typing
nox -s unit
uv run --group=nox nox -s lint
uv run --group=nox nox -s typing
uv run --group=nox nox -s unit
With unit tests, you can run a specific test file or directory:
nox -s test -- tests/foo_test.py
uv run --group=nox nox -s test -- tests/foo_test.py
To see a listing of specific nox sessions, run:
nox --list
uv run --group=nox nox --list
Building documentation¶
Documentation is built with Sphinx:
nox -s docs
uv run --group=nox nox -s docs
The built documentation is located in the docs/_build/html
directory.
To check the documentation for broken links, run:
nox -s docs-linkcheck
uv run --group=nox nox -s docs-linkcheck
Updating the change log¶
Ook uses scriv to maintain its change log.
When preparing a pull request, run
nox -s scriv-create
uv run --group=nox nox -s scriv-create
This will create a change log fragment in changelog.d
.
Edit that fragment, removing the sections that do not apply and adding entries for your pull request.
Change log entries use the following sections:
Backward-incompatible changes
New features
Bug fixes
Other changes (for minor, patch-level changes that are not bug fixes, such as logging formatting changes or updates to the documentation)
Do not include a change log entry solely for updating pinned dependencies, without any visible change to Ook’s behavior. Every release is implicitly assumed to update all pinned dependencies.
These entries will eventually be cut and pasted into the release description for the next release, so the Markdown for the change descriptions must be compatible with GitHub’s Markdown conventions for the release description. Specifically:
Each bullet point should be entirely on one line, even if it contains multiple sentences. This is an exception to the normal documentation convention of a newline after each sentence. Unfortunately, GitHub interprets those newlines as hard line breaks, so they would result in an ugly release description.
Avoid using too much complex markup, such as nested bullet lists, since the formatting in the GitHub release description may not be what you expect and manually editing it is tedious.
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.