Skip to content

Latest commit

History

History
100 lines (71 loc) 路 3.76 KB

CONTRIBUTING.md

File metadata and controls

100 lines (71 loc) 路 3.76 KB

Contributing guide

Thank you for your interest in contributing to the Rust In Pieces project!

There are a few ways of helping: critiquing the documentation and/or code examples, fixing incorrect information, fixing bugs, and fixing errors in the book.

Contributions can be made in two places: the code and the book.

Code

The code is in the pieces directory.

It is encouraged for you to run the tests, formatters, and linters before submitting PRs. They will all need to pass before the PR can be merged so you can save youself some time by running them locally before submitting. The README of each piece contains instructions for running these checks.

Additionally we use pre-commit to run linting checks. To install pre-commit create a virtual environment, activate it, and then install the requirements (Note - if you have pre-commit installed at your system level you can skip this step):

# First time setup
python -m venv venv
source venv/bin/activate
pip install -r requirements-dev.txt

# For subsequent runs, simply activate the environment
source venv/bin/activate

Next install the pre-commit hooks:

pre-commit install

Now any time you commit your code pre-commit will run its checks. If you want to run the checks before commiting you can do so manually:

pre-commit run --all-files

If committing Rust code, make sure to run Cargo's clippy and fmt on the piece so that linting is respected. You can do this by either using the Makefile provided in each piece, or by running the Cargo commands individually.

make all

# Runs the following
cargo fmt --all --quiet
cargo check --all-targets --quiet
cargo clippy --all-targets --quiet
cargo test --quiet
cargo run --quiet

Critiquing the documentation and/or code

This is the easiest way to contribute. Basically, as you read the documentation or experiment with the code samples, if you find something confusing, incorrect, or missing, then you can file an issue explaining your concerns.

Fixing incorrect information

If you find typos or incorrect links we welcome PRs to fix them. For these types of fixes there is no need to open an issue first. For larger changes (or potentially adding a new piece), you are encouraged to first open an issue to discuss the change before taking the time to make changes.

Fixing bugs

If you find a bug in the code examples as you work through them, PRs for fixes are welcome. When fixing a bug in either a Python or Rust file please check the other language's corresponding file to see if the same bug exists there. If you know how to fix both then you can do both in one PR. If you are unsure how to make the fix in one language let us know in your PR description so we can make the additional fix.

Adding a new piece (maybe)

It's recommended to start a discussion prior to putting in the work to add a new piece. This will help ensure that the piece is a good fit for the project and that there is interest in it.

Book

The book is written using mdbook, a minimalist Rust-based framework to write books in Markdown. It's hosted on GitHub Pages and is available at rustinpieces.dev.

Fixing errors in the book

If you find an error in the book's text or code blocks, please open a PR with the fix. If you are unsure how to fix the error, please open an issue instead.

It's strongly recommended to build and test locally prior to pushing any fixes. See the book's README.md for instructions on how to build and run the book locally.