76040bfa87
Add information about how the continuous integration is set up in Dendrite and how to run the tests locally so that people don't need to wait around for things to churn.
83 lines
3.7 KiB
Markdown
83 lines
3.7 KiB
Markdown
# Contributing to Dendrite
|
|
|
|
Everyone is welcome to contribute to Dendrite! We aim to make it as easy as
|
|
possible to get started.
|
|
|
|
Please ensure that you sign off your contributions! See [Sign Off](#sign-off)
|
|
section below.
|
|
|
|
## Getting up and running
|
|
|
|
See [INSTALL.md](INSTALL.md) for instructions on setting up a running dev
|
|
instance of dendrite, and [CODE_STYLE.md](CODE_STYLE.md) for the code style
|
|
guide.
|
|
|
|
As of May 2019, we're not using `gb` anymore, which is the tool we had been
|
|
using for managing our dependencies. We're now using Go modules. To build
|
|
Dendrite, run the `build.sh` script at the root of this repository (which runs
|
|
`go install` under the hood), and to run unit tests, run `go test ./...` (which
|
|
should pick up any unit test and run it). There are also [scripts](scripts) for
|
|
[linting](scripts/find-lint.sh) and doing a [build/test/lint
|
|
run](scripts/build-test-lint.sh).
|
|
|
|
## Continuous Integration
|
|
|
|
When a Pull Request is submitted, continuous integration jobs are run
|
|
automatically to ensure the code builds and is relatively well-written. Checks
|
|
are run on [Buildkite](https://buildkite.com/matrix-dot-org/dendrite/) and
|
|
[CircleCI](https://circleci.com/gh/matrix-org/dendrite/).
|
|
|
|
If a job fails, click the "details" button and you should be taken to the job's
|
|
logs.
|
|
|
|
|
|
|
|
Scroll down to the failing step and you should see some log output. Scan
|
|
the logs until you find what it's complaining about, fix it, submit a new
|
|
commit, then rinse and repeat until CI passes.
|
|
|
|
### Running CI Tests Locally
|
|
|
|
To save waiting for CI to finish after every commit, it is ideal to run the
|
|
checks locally before pushing, fixing errors first. This also saves other
|
|
people time as only so many PRs can be tested at a given time.
|
|
|
|
To execute what Buildkite tests, simply run `./scripts/build-test-lint.sh`.
|
|
This script will build the code, lint it, and run `go test ./...` with race
|
|
condition checking enabled. If something needs to be changed, fix it and then
|
|
run the script again until it no longer complains. Be warned that the linting
|
|
can take a significant amount of CPU and RAM.
|
|
|
|
CircleCI simply runs [Sytest](https://github.com/matrix-org/sytest) with a test
|
|
whitelist. See
|
|
[docs/sytest.md](https://github.com/matrix-org/dendrite/blob/master/docs/sytest.md#using-a-sytest-docker-image)
|
|
for instructions on setting it up to run locally.
|
|
|
|
|
|
## Picking Things To Do
|
|
|
|
If you're new then feel free to pick up an issue labelled [good first issue](https://github.com/matrix-org/dendrite/labels/good%20first%20issue).
|
|
These should be well-contained, small pieces of work that can be picked up to
|
|
help you get familiar with the code base.
|
|
|
|
Once you're comfortable with hacking on Dendrite there are issues lablled as
|
|
[help wanted](https://github.com/matrix-org/dendrite/labels/help%20wanted), these
|
|
are often slightly larger or more complicated pieces of work but are hopefully
|
|
nonetheless fairly well-contained.
|
|
|
|
We ask people who are familiar with Dendrite to leave the [good first issue](https://github.com/matrix-org/dendrite/labels/good%20first%20issue)
|
|
issues so that there is always a way for new people to come and get involved.
|
|
|
|
## Getting Help
|
|
|
|
For questions related to developing on Dendrite we have a dedicated room on
|
|
Matrix [#dendrite-dev:matrix.org](https://matrix.to/#/#dendrite-dev:matrix.org)
|
|
where we're happy to help.
|
|
|
|
For more general questions please use [#dendrite:matrix.org](https://matrix.to/#/#dendrite:matrix.org).
|
|
|
|
## Sign off
|
|
|
|
We ask that everyone who contributes to the project signs off their
|
|
contributions, in accordance with the [DCO](https://github.com/matrix-org/matrix-doc/blob/master/CONTRIBUTING.rst#sign-off).
|