Skip to content

Commit b309f4a

Browse files
Apply suggestions from code review
Co-authored-by: davidlion <[email protected]>
1 parent 61bc125 commit b309f4a

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

README.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,15 +44,14 @@ task build:unit-test-<lib_name>
4444
```
4545

4646
## Testing
47-
To build and run unit tests for a specific library:
47+
To build and run all unit tests:
4848
```shell
49-
task test-<lib_name>
49+
task test-all
5050
```
5151

52-
#### Special Targets
53-
To build and run all unit tests:
52+
To build and run unit tests for a specific library:
5453
```shell
55-
task test-all
54+
task test-<lib_name>
5655
```
5756

5857
## Linting

taskfile.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ tasks:
2929
- "mkdir -p '{{.G_BUILD_DIR}}'"
3030

3131
test-*:
32-
desc: "Runs unit tests for the specified library. See ystdlib-cpp README for special targets."
32+
desc: "Runs unit tests for the specified test target."
3333
vars:
3434
TEST_TARGET: >-
3535
{{printf "unit-test-%s" (index .MATCH 0)}}

taskfiles/build.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ tasks:
3838
{{printf "unit-test-%s" .TARGET_SUFFIX}}
3939
preconditions:
4040
- sh: >-
41-
{{(has .TARGET_SUFFIX .G_TEST_TARGET_SUFFIXES)}}
41+
{{has .TARGET_SUFFIX .G_TEST_TARGET_SUFFIXES}}
4242
msg: |-
4343
{{.TARGET_NAME}} is not a valid unit test target!
4444
List of unit test targets:

0 commit comments

Comments
 (0)