Fix the docker tests instructions (#543)

* Tested on multiple hardware

Turns out windows 10 and 11 can't use `$(pwd):/app` OR `%cd%:/app` in PowerShell (which replaced cmd prompt), so I looked up the docs and made the necessary changes (again, sorry!)

These changes have been tested on all terminal environments except macOS (the docs say it'll work there)

* allow use of --exitfirst for faster debug

* Remove install in actions

---------

Co-authored-by: Bas Nijholt <basnijholt@gmail.com>
Co-authored-by: Bas Nijholt <bas@nijho.lt>
This commit is contained in:
Benjamin Auquite 2023-04-05 11:13:57 -05:00 committed by GitHub
commit f5abf034c4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -5,17 +5,15 @@ Alternatively, you can use the provided Docker image to run the tests locally.
To run the tests using the Docker image, navigate to the `adaptive-lighting` repo folder and execute the following command:
Linux or MacOS:
Linux / MacOS / Windows PowerShell:
```bash
docker run -v $(pwd):/app basnijholt/adaptive-lighting:latest
docker run -v ${PWD}:/app basnijholt/adaptive-lighting:latest
```
Windows:
```bash
docker run -v %cd%:/app basnijholt/adaptive-lighting:latest
```
- In windows command prompt, the command is:
```bash
docker run -v %cd%:/app basnijholt/adaptive-lighting:latest
```
This command will download the Docker image from [the adaptive-lighting Docker Hub repo](https://hub.docker.com/r/basnijholt/adaptive-lighting) and run the tests.