Update README/CONTRIBUTING/sytest for AWSY issues (#1333)

* Update README/CONTRIBUTING/sytest for AWSY issues

* More docs

2 files changed, 41 insertions, 16 deletions

diff --git a/docs/CONTRIBUTING.md b/docs/CONTRIBUTING.md
index 72a588af..30ef65ea 100644
--- a/docs/CONTRIBUTING.md
+++ b/docs/CONTRIBUTING.md
@@ -12,18 +12,18 @@ 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 [build/scripts](/build/scripts) for
-[linting](/build/scripts/find-lint.sh) and doing a [build/test/lint
-run](/build/scripts/build-test-lint.sh).
-
-As of February 2020, we are deprecating support for Go 1.11 and Go 1.12 and are
-now targeting Go 1.13 or later. Please ensure that you are using at least Go
-1.13 when developing for Dendrite - our CI will lint and run tests against this
-version.
+We use [golangci-lint](https://github.com/golangci/golangci-lint) to lint
+Dendrite which can be executed via:
+
+```
+$ golangci-lint run
+```
+
+We also have unit tests which we run via:
+
+```
+$ go test ./...
+```
 
 ## Continuous Integration
 
@@ -73,7 +73,7 @@ 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),
+[help wanted](https://github.com/matrix-org/dendrite/labels/help-wanted),
 these are often slightly larger or more complicated pieces of work but are
 hopefully nonetheless fairly well-contained.
 
diff --git a/docs/sytest.md b/docs/sytest.md
index 26a46c0b..03954f13 100644
--- a/docs/sytest.md
+++ b/docs/sytest.md
@@ -10,9 +10,9 @@ passes.
 
 ## Finding out which tests to add
 
-We recommend you run the tests locally by using the SyTest docker image or
-manually setting up SyTest. After running the tests, a script will print the
-tests you need to add to `sytest-whitelist`.
+We recommend you run the tests locally by using the SyTest docker image.
+After running the tests, a script will print the tests you need to add to
+`sytest-whitelist`.
 
 You should proceed after you see no build problems for dendrite after running:
 
@@ -20,8 +20,16 @@ You should proceed after you see no build problems for dendrite after running:
 ./build.sh
 ```
 
+If you are fixing an issue marked with
+[Are We Synapse Yet](https://github.com/matrix-org/dendrite/labels/are-we-synapse-yet)
+then there will be a list of Sytests that you should add to the whitelist when you
+have fixed that issue. This MUST be included in your PR to ensure that the issue
+is fully resolved.
+
 ### Using the SyTest Docker image
 
+**We strongly recommend using the Docker image to run Sytest.**
+
 Use the following commands to pull the latest SyTest image and run the tests:
 
 ```sh
@@ -38,12 +46,29 @@ When debugging, the following Docker `run` options may also be useful:
 * `-v /path/to/sytest/:/sytest/`: Use your local SyTest repository at
   `/path/to/sytest` instead of pulling from GitHub. This is useful when you want
   to speed things up or make modifications to SyTest.
+* `-v "/path/to/gopath/:/gopath"`: Use your local `GOPATH` so you don't need to
+  re-download packages on every run.
 * `--entrypoint bash`: Prevent the container from automatically starting the
   tests.  When used, you need to manually run `/bootstrap.sh dendrite` inside
   the container to start them.
+* `-e "DENDRITE_TRACE_HTTP=1"`: Adds HTTP tracing to server logs.
+* `-e "DENDRITE_TRACE_INTERNAL=1"`: Adds roomserver internal API tracing to
+  server logs.
+* `-e "DENDRITE_TRACE_SQL=1"`: Adds tracing to all SQL statements to server logs.
+
+The docker command also supports a single positional argument for the test file to
+run, so you can run a single `.pl` file rather than the whole test suite. For example:
+```
+docker run --rm --name sytest -v "/Users/kegan/github/sytest:/sytest"
+-v "/Users/kegan/github/dendrite:/src" -v "/Users/kegan/logs:/logs"
+-v "/Users/kegan/go/:/gopath" -e "POSTGRES=1" -e "DENDRITE_TRACE_HTTP=1"
+matrixdotorg/sytest-dendrite:latest tests/50federation/40devicelists.pl
+```
 
 ### Manually Setting up SyTest
 
+**We advise AGAINST using manual SyTest setups.**
+
 If you don't want to use the Docker image, you can also run SyTest by hand. Make
 sure you have Perl 5 or above, and get SyTest with: