Contributing¶
Thank you for your interest in contributing to mp-units! 👍🎉
Before contributing, we strongly recommend familiarizing yourself with the official documentation.
This file is a part of MkDocs documentation
This file is part of the documentation and uses some MkDocs‑specific formatting that may look unusual in a plain Markdown renderer. For the fully rendered version, visit the Contributing chapter.
It is symlinked into the documentation as docs/getting_started/contributing.md, so links
to other repository files have to be written as absolute https://github.com/mpusz/mp-units
URLs. A repository-relative link would resolve against the documentation tree and break on
the website.
Where to Start¶
- See good first issues for beginner-friendly tasks.
- help wanted and high priority labels highlight other important work.
Preconfigured Development Environment¶
The easiest way to start coding is to use our containerized development environment in GitHub Codespaces. You can either click the button below
or navigate to the main page of the repository and click "Code" button -> "Codespaces" tab -> "Create codespace on master".
You can also use the pre-configured devcontainer and Docker image manually within your IDE.
Setting Up Your Codespace¶
- Fork the repository on GitHub (if you haven't already)
-
Create a CodeSpace from your fork
- Navigate to your forked repository
- Click "Code" → "Codespaces" → "Create codespace on master"
- Choose a 4-core machine or larger for better performance
-
Wait for initialization (~3–5 minutes)
- Container will download and start automatically
- Conan will be configured for your environment
- All VS Code extensions will be installed
Helpful commands¶
# Test with a single compiler configuration
conan build . -pr gcc15 -c user.mp-units.build:all=True -b missing
# Build and test with all supported compilers
.devcontainer/check_all.sh build
# ... or build them as Conan packages instead
.devcontainer/check_all.sh create
# Generating API Reference
.devcontainer/api_reference.sh
# Preview documentation locally
mkdocs serve
# Run pre-commit checks
pre-commit run --all-files
For more, see the official documentation and
.devcontainer/README.md.
Tips¶
- Use the provided scripts for building, testing, and packaging. See the docs for advanced Conan/CMake options.
- For documentation, see the Material for MkDocs guide.
- Run
.devcontainer/check_all.sh buildto test across all supported compilers (the script requires one ofinstall,build, orcreate; run it with-hfor the options). - Use
pre-committo auto-format code and check style before committing. - See the C++ compiler support page for supported toolchains.
Alternative: Custom Development Environment
Custom Development Environment¶
Alternatively, please refer to our official docs for download, build, and install instructions with the below changes if you want to set up a development environment on your local machine.
Building the entire repository¶
To build all the mp-units source code (with unit tests and examples), you should:
- Use the CMakeLists.txt from the top-level directory.
- Run Conan with
user.mp-units.build:all=True.
git clone https://github.com/mpusz/mp-units.git && cd mp-units
conan build . -pr <your_conan_profile> -s compiler.cppstd=23 -c user.mp-units.build:all=True -b missing
The above will download and install all of the dependencies needed for the development of the library, build all of the source code, and run unit tests.
If you prefer to build the project via CMake rather than Conan, then you should replace
the conan build with conan install command and then follow with a regular CMake build and testing:
conan install . -pr <your_conan_profile> -s compiler.cppstd=23 -c user.mp-units.build:all=True -b missing
cmake --preset conan-default
cmake --build --preset conan-release
cmake --build --preset conan-release --target all_verify_interface_header_sets
cmake --build --preset conan-release --target test
Hint
To ensure that we always build all the targets and to save some typing of the Conan commands,
we can set the following in the ~/.conan2/global.conf:
Packaging¶
To test CMake installation and Conan packaging run:
conan create . --user <username> --channel <channel> -pr <your_conan_profile> -s compiler.cppstd=23 \
-c user.mp-units.build:all=True -b missing
The above will create a Conan package and run tests provided in ./test_package directory.
In case you would like to upload mp-units package to the Conan server, do the following:
Building documentation¶
We are building our documentation using Material for MkDocs.
The easiest way to install all the required dependencies is with pip and the
requirements.txt file
provided in the repository:
Additionally, a Cairo Graphics library is required by Material for MkDocs. Please follow the official MkDocs documentation to install it.
After that, you can either:
Generating API reference¶
We need to take a few steps to set up our environment so that we are ready to generate API reference documents.
First, we need to satisfy the requirements described in https://github.com/cplusplus/draft and https://github.com/Eelis/cxxdraft-htmlgen. On the Ubuntu platform, this is equivalent to the following instructions run from the user's home directory:
sudo apt install latexmk texlive-latex-recommended texlive-latex-extra texlive-fonts-recommended lmodern
sudo apt install haskell-stack graphviz nodejs npm ghc cabal-install
npm install split mathjax-full mathjax-node-sre mathjax-node-cli yargs@16.2.0
cabal update
On some platforms, installing mathjax-node-cli through npm does not update the system's PATH
environment variable resulting in tex2html not found errors. In such cases we need to add
the .bin folder to the PATH environment variable manually:
Now, we are ready to start building our API reference using our automated script:
This script will:
- Configure CMake with the correct module paths
- Build both PDF and HTML documentation
- Create a symlink to the generated documentation
If you only want to setup CMake without building the documentation, use:
Custom Dependency Directory¶
By default, the script downloads dependencies to ../api_reference_deps relative to the
project root. You can specify a custom directory using the -d parameter:
# Use a custom directory for dependencies
.devcontainer/api_reference.sh -d /path/to/custom/deps
# Combine with setup-only mode
.devcontainer/api_reference.sh -s -d ./local_deps
The dependency directory will contain:
jegp.cmake_modules: CMake modules for documentation generationdraft: C++ standard draft sources for reference formattingcxxdraft-htmlgen: HTML generation tools for standardese-style output
This is useful when you want to:
- Share dependencies across multiple project checkouts
- Avoid re-downloading dependencies for development containers
Manual documentation generation¶
Alternatively, you can run the individual steps manually.
First, we need to clone the following git repositories:
- https://github.com/JohelEGP/jegp.cmake_modules
standardese_sources_basebranch of https://github.com/JohelEGP/draftstandardese_sources_basebranch of https://github.com/JohelEGP/cxxdraft-htmlgen
For example:
git clone https://github.com/JohelEGP/jegp.cmake_modules.git --depth=1
git clone https://github.com/JohelEGP/draft.git --branch=standardese_sources_base --depth=1
git clone https://github.com/JohelEGP/cxxdraft-htmlgen.git --branch=standardese_sources_base --depth=1
Next, configure CMake with the following:
cmake -S docs/api_reference/src -B build/docs/api_reference \
-DCMAKE_MODULE_PATH="<path to gh:JohelEGP/jegp.cmake_modules>/modules" \
-DJEGP_STANDARDESE_SOURCES_GIT_REPOSITORY="<path to gh:JohelEGP/draft>" \
-DJEGP_CXXDRAFT_HTMLGEN_GIT_REPOSITORY="<path to gh:JohelEGP/cxxdraft-htmlgen>"
Then build the documentation with CMake:
The generated documentation will be available at:
- PDF:
build/docs/api_reference/mp-units.pdf - HTML:
build/docs/api_reference/mp-units.html/
In the end, we need to move the generated documentation to the docs/api_reference/gen subdirectory:
or just link the entire directory:
Configuration Options for Developers and Contributors¶
Conan¶
user.mp-units.build:all-
Enables compilation of all the source code, including tests and examples. To support this, it requires some additional Conan build dependencies described in Repository directory tree and dependencies. It also runs unit tests during the Conan build (unless
tools.build:skip_testconfiguration property is set toTrue).
user.mp-units.analyze:clang-tidy-
Enables clang-tidy analysis.
CMake¶
-
2.2.0 ·
ON/OFF(Default:OFF)Enables include-what-you-use analysis.
-
2.2.0 ·
ON/OFF(Default:OFF)Enables clang-tidy analysis.
-
2.5.0 ·
NONE/ALL/MODULES/HEADERS(Default:NONE)Enables compilation performance data collection with
-ftime-tracefor clang compilers.All our unit tests compile only for headers and never for modules. To allow fair comparison,
MODULESandHEADERSdo not enable the data collection for unit tests. This means that they affect only the core, systems, and examples.Please use
ALLto profile unit tests as well.
AI-Generated Contributions¶
We welcome AI assistants and coding agents (LLMs) to help you write prose, generate tests, explore a design, or draft a change. They are useful tools. But mp-units is a bleeding-edge C++20/23/26 library, and the bar we hold code to does not drop because a machine wrote it. Whatever tool you use, you are the author: you are accountable for every line, and we review it as your work.
- Understand and defend every line: Never open a "blind" pull request. If you cannot explain why a change is correct and why it is designed the way it is, it is not ready, however confident the model sounded.
- Build and test it yourself first: Run the change in the provided
.devcontainer(or GitHub Codespaces) and make the tests pass locally, rather than leaving that to CI or to the reviewer. AI-written tests are especially suspect: a test must encode the intended behavior, not lock in whatever the code currently happens to do. - Keep the diff minimal: No unrequested reformatting, renaming, or file-wide rewrites. Large, monolithic diffs for a small change will be rejected.
- Watch for confident-but-wrong code: Models are weakest exactly where this library lives, on the newest language features, template metaprogramming, and compile-time interfaces. Check for legacy patterns, needless template machinery, and subtly incorrect metaprogramming.
- Be transparent: If a coding agent or an LLM did a significant part of the work, say so in the pull request, so reviewers know where to look hardest.
Low-effort, untested, or high-volume agent-generated pull requests, submitted faster than a human could have reviewed them, will be closed without a long back-and-forth. A contribution is a request for our time; please make sure it is worth more of ours than it cost you.
If you drive your contribution with an AI coding agent, point it at
AGENTS.md, which gives the agent
the build, test, and design pointers it needs to follow these rules.
Before committing git changes¶
There are a few recommended checks before committing and pushing changes.
Naming conventions¶
Here are the main rules for naming things in this repo:
- types, functions, variables use
standard_case, - template parameters use
PascalCase, - C++ concepts:
PascalCase(subject to change)
Avoid one-letter lowercase names for variables and parameters. The library defines very short
unit symbols (m, s, l, h, ...), so such a name easily collides with one of them, and
MSVC then reports a shadowing warning that no other compiler produces. Prefer descriptive
names, and lhs/rhs for the two sides of a binary operator. Short uppercase template
parameter names are fine.
Code style¶
Formatting is handled for you by pre-commit, but a few conventions are worth knowing before
you write mp-units code:
- Prefer CTAD to
autofor the library's own types —quantity q = 42 * m;rather thanauto q = 42 * m;. The type name carries semantic weight, and hiding it costs the reader more than the typing saves. A constrainedQuantityOf<isq::length> autois a reasonable middle ground when the type is genuinely not evident from the right-hand side. Plainautoremains fine for standard library types such as iterators. - Prefer copy initialization (
T var = expr;) to direct initialization (T var{expr};). Copy initialization considers only implicit conversions, so the syntax itself records whether a conversion was meant to be implicit; braces quietly accept explicit ones too. - Comments explain why, not what. A comment narrating what the following block does usually means that block wants to be a named function, or that a condition wants to be a named concept; extract it and drop the comment. Keep the comments no name can carry — compiler-bug workarounds, standard-mandated subtleties, invariants, and references to a specification.
Unified code formatting¶
A formatting standard is enforced with the pre-commit script. Before committing your changes,
please do the following:
This will run:
clang-formatfor code formatting with the.clang-formatfile provided in the repo,cmake-formatto format the CMake files,markdownlint-cli2for the Markdown documentation (MD013caps the line length, so prose has to be wrapped),shellcheckfor the shell scripts andactionlintfor the GitHub Actions workflows,black,isort, andflake8for the Python scripts,- some other checks (e.g., trailing whitespace, YAML and JSON validity, etc.).
The script will run on all the files in the repo and will apply the changes in place when needed. After it finishes review and stage all modified files.
Header placement¶
Everything the library exports lives in the public directories. A header under a bits/
subdirectory holds implementation details only and must not export anything: no
MP_UNITS_EXPORT, no MP_UNITS_EXPORT_BEGIN. If an entity has to be visible to users, or
has to be named by an exported declaration, its header belongs in a public directory such
as framework/ instead.
Note that this is about the directory, not the namespace. An entity may perfectly well stay
in mp_units::detail while living in a public header, because being reachable to the module's
importers and being part of the supported API are two different things.
Relatedly, never place an #include in the middle or at the end of a header. When
splitting a large header up, decompose it into self-contained components in a linear
dependency chain plus a thin aggregator whose includes all sit at the top, keeping the
public aggregator path stable so that existing include sites are unaffected.
Backward compatibility¶
Before submission, please remember to check if the code compiles fine on the supported compilers. The CI will check it anyway, but it is good to check at least some of the configurations before pushing changes. Especially older compilers can be tricky as those do not have full C++20 conformance. The official list of supported compilers can always be found in the C++ compiler support (API/ABI) chapter of our documentation.
CI/CD Integration¶
Each pull request triggers a comprehensive CI pipeline:
- Multi-compiler builds: GCC 12-16, Clang 16-21 except Clang-19, Apple Clang, and MSVC, across C++20/23/26 and the library's configuration options
- Code formatting: clang-format and pre-commit checks
- Static analysis: clang-tidy analysis for code quality
- Documentation: Verification that docs build successfully
- Package testing: Conan package creation and consumption tests
- Dependency security: Automated vulnerability and license scanning
The compiler matrix is sampled, not exhaustive
The full combination of toolchains, C++ versions, and library options is far too large to
build on every push, so
.github/generate-job-matrix.py
selects a random subset for each run (while guaranteeing coverage of the configurations
that change the API). A green pull request therefore does not prove that every
configuration builds, and a later run may uncover a problem in a combination yours never
tried. This is expected, and not a sign that something went wrong with your change.
For Maintainers: Creating Good First Issues¶
Guidance for Repository Maintainers
This section provides guidance for maintainers creating curated onboarding tasks.
A "good-first-issue" label alone is insufficient. Well-crafted good first issues remove decision paralysis and provide explicit mentorship commitment. They are invitations, not just labels.
When creating a good first issue, use the [MAINTAINERS ONLY] Good First Issue Template in the issue picker. This structured template ensures you include:
- What: Specific deliverables with file paths and clear scope
- Why: Rationale for the work and its impact on the project
- Mentorship: Explicit commitment to guide the contributor through the PR
- Skills Learned: Concrete concepts, APIs, or techniques they'll learn
- Estimated Time: Realistic time investment (1-8 hours)
- Getting Started: Optional step-by-step guidance
- Acceptance Criteria: Optional completion checklist
Example¶
## Good First Issue: Add Celsius to Fahrenheit example
**What:** Create a new example in `example/temperature_conversion.cpp` that demonstrates:
- Converting between Celsius and Fahrenheit using affine space
- Using `quantity_point` for absolute temperatures
- Using `quantity` for temperature differences
**Why:** Temperature conversion is a common use case that confuses new users.
This example will clarify the distinction between absolute and relative temperatures.
**Mentorship:** @mpusz will help you through the PR process
**Skills Learned:**
- C++20 concepts and constraints
- mp-units API design patterns
- Affine space modeling
- Writing clear example code
**Estimated Time:** 2 hours
Philosophy¶
Lower the barrier to entry with clear guidance. A well-written good-first-issue provides:
- Zero ambiguity — contributor knows exactly what to do and where
- Context — understands why the work matters
- Safety net — explicit promise that someone will guide them
- Growth — clear learning objectives make it worthwhile
This is not just labeling issues. This is curated mentorship at scale.