Anyone is welcome to contribute to Revault regardless of experience, age, or any other concern. However, Bitcoin development (and especially vaults for that matter) requires a high level of rigor, so it could take some time (and backs and forths) to polish a contribution before it's ready for merge.
Most of the communication is done on GitHub (we'll setup more channels as the project grows).
Don't hesitate to open issues to ask questions or conceptually discuss a change before going forward with the implementation if it is non-trivial.
The codebase is maintained using the "contributor workflow" where everyone without exception contributes patch proposals using "pull requests" (PRs). This facilitates social contribution, easy testing and peer review.
In general, commits should be atomic and diffs should be easy to read. For this reason, do not mix any formatting fixes or code moves with actual code changes.
When possible, make sure each individual commit is hygienic: that it builds successfully on its own without warnings, errors, regressions, or test failures.
Commit messages should be verbose by default consisting of a short subject line, a blank line and detailed explanatory text as separate paragraph(s), unless the title alone is self-explanatory. Commit messages should be helpful to people reading your code in the future, so explain the reasoning for your decisions.
If your pull request contains fixup commits (commits that change the same line of code repeatedly) or too fine-grained commits, you may be asked to squash your commits before it will be merged.
Patchsets should always be focused. For example, a pull request could add a feature, fix a bug, or refactor code; but not a mixture. Please also avoid super pull requests which attempt to do too much, are overly large, or overly complex as this makes review difficult. Instead, prefer opening different focused pull requests.
Anyone may participate in peer review which is expressed by comments in the pull request. Typically reviewers will review the code for obvious errors, as well as test out the patch set and opine on the technical merits of the patch. PR should be reviewed first on the conceptual level before focusing on code style or grammar fixes.
We test message processing against Postgre. Therefore the tests need a running instance
of Postgre to connect to, postgresql://revault:revault@localhost/coordinator_db
. The easiest
way to set one up is by using docker:
docker run --rm -d -p 5432:5432 --name postgres-coordinatord -e POSTGRES_PASSWORD=revault -e POSTGRES_USER=revault -e POSTGRES_DB=coordinator_db postgres:alpine
cargo test
You'll need honggfuzz-rs for running fuzz tests.
Also, you'll need a running Postgres database. The tests will look for a database at
postgresql://revault:revault@localhost:5432/fuzz_coordinator
, but you can modify
the uri using the POSTGRES_URI
environment variable.
cd fuzz/ && cargo hfuzz run send_msg
You should definitely clone the existing corpus for the send_msg
target and start extending it from there. For example:
# Still from the root of the fuzz/ directory
git clone https://github.com/revault/coordinatord_fuzz_corpus
HFUZZ_RUN_ARGS="--exit_upon_crash -v --input coordinatord_fuzz_corpus" cargo hfuzz run send_msg
Refer to Honggfuzz's doc for more run options.
To avoid endless bikeshedding, just use rustfmt
.
Clippy is also often your friend.