Contributing¶
Contributions are welcome, and they are greatly appreciated! Every little bit helps, and credit will always be given.
Types of Contributions¶
You can contribute in many ways:
Report Bugs¶
Report bugs at https://github.com/bpcreech/PyMiniRacer/issues.
If you are reporting a bug, please include:
- Your operating system name and version.
- Any details about your local setup that might be helpful in troubleshooting.
- Detailed steps to reproduce the bug.
Fix Bugs¶
Look through the GitHub issues for bugs. Anything tagged with "bug" is open to whoever wants to implement it.
Implement Features¶
Look through the GitHub issues for features. Anything tagged with "feature" is open to whoever wants to implement it.
Write Documentation¶
Python Mini Racer could always use more documentation, whether as part of the official Python Mini Racer docs, in docstrings, or even on the web in blog posts, articles, and such.
Submit Feedback¶
The best way to send feedback is to file an issue at https://github.com/bpcreech/PyMiniRacer/issues.
If you are proposing a feature:
- Explain in detail how it would work.
- Keep the scope as narrow as possible, to make it easier to implement.
- Remember that this is a volunteer-driven project, and that contributions are welcome :)
Get Started!¶
Ready to contribute? Here's how to set up PyMiniRacer
for local development.
Warning
Building this package from source takes several GB of disk space and takes 1-2 hours.
-
Do a quick scan through the architecture guide before diving in.
-
Fork the
PyMiniRacer
repo on GitHub. -
If you plan to change C++ code you should probably install at least
clang-format
andclang-tidy
from the latest stable LLVM. While thePyMiniRacer
build uses its own compiler (!) on most systems, our pre-commit rules rely on the systemclang-format
andclang-tidy
. If your versions of those utilities do not match the onesPyMiniRacer
uses on GitHub Actions, you may see spurious pre-commit errors.If you're on a Debian-related Linux distribution using the LLVM project's standard apt packages, note that you will likely have to override
/usr/bin/clang-format
and/usr/bin/clang-tidy
to point to the latest version, i.e.,/usr/bin/clang-format-18
and/usr/bin/clang-tidy-18
, respectively.You can always silence local pre-commit errors with the
-n
argument togit commit
. We checkpre-commit
s on every pull request using GitHub Actions, so you can check for errors there instead. -
Run some of the following:
# Set up a virtualenv: $ python -m venv ~/my_venv $ . ~/my_venv/bin/activate $ pip install pre-commit hatch hatch-mkdocs # Set up a local clone of your fork: $ git clone git@github.com:your_name_here/PyMiniRacer.git $ cd PyMiniRacer/ $ pre-commit install # install our pre-commit hooks # Build and test stuff: $ hatch run docs:serve # build the docs you're reading now! $ hatch build # this may take 1-2 hours! $ hatch run test:run
You can also play with your build in the Python REPL, as follows:
$ hatch shell $ python >>> from py_mini_racer import MiniRacer >>> mr = MiniRacer() >>> mr.eval('6*7') 42 >>> exit() $ exit
As a shortcut for iterative development, you can skip the
hatch build
and matrix tests, and do: -
Create a branch for local development::
Now you can make your changes locally.
-
When you're done making changes, check that your changes pass the linter and the tests, including testing other Python versions:
-
Commit your changes and push your branch to GitHub::
-
(Optional) Run the GitHub Actions build workflow on your fork to ensure that all architectures work.
-
Submit a pull request through the GitHub website.
Tests¶
If you want to run the tests, you need to build the package first:
Then run:
Or for the full test matrix:
Pull Request Guidelines¶
Before you submit a pull request, check that it meets these guidelines:
- The pull request should include tests.
- If the pull request adds functionality, the docs should be updated. Put your new functionality into a function with a docstring, and add the feature to the list in README.md.
- The pull request should work for the entire test matrix of Python versions
(
hatch run tests:run
).
Releasing PyMiniRacer
¶
Releases for PyMiniRacer
should be done by GitHub Actions on the official project
repository.
Ordinary releases¶
To make an ordinary release from main
:
-
Merge all changes into
main
on the official repository. -
Pick the next revision number:
-
Create a
feature/...
branch, and:-
Update
HISTORY.md
with a summary of changes since the last release. -
Update
src/py_mini_racer/__about__.py
with the new revision number. -
Create and merge a pull request for this branch into
main
.
-
-
Create a
release/...
branch: -
Observe the build process on GitHub Actions. It should build and push docs and upload wheels to PyPI automatically.
Warning
As of this writing, the
aarch64
Linux builds are slow because they're running on emulation. They time out on the first try (and second and third and...) after 6 hours. If you "restart failed jobs", they will quickly catch up to where where they left off due tosccache
. The jobs should eventually complete within the time limit. You can observe their slow progress using the Ninja build status (e.g.,[1645/2312] CXX obj/v8_foo_bar.o
).
Hotfix releases¶
To hotfix a prior release:
-
Prepare the fix as a
feature
branch as normal, and merge it intomain
. -
Pick the next revision number. This will typically be a patch-version update on the current release name.
-
Create a new release branch for the hotfix:
-
Hotfix the commit(s) created in step #1 onto the new release branch.
-
Create a version update commit:
-
Update
HISTORY.md
with a summary of changes since the last release. -
Update
src/py_mini_racer/__about__.py
with the new revision number.
-
-
Commit and push the new release branch to GitHub.
-
Merge this branch into
main
. All content on release branches should be included inmain
. -
Observe the build process on GitHub Actions. It should build and push docs and upload wheels to PyPI automatically.