Skip to content

Commit 2337553

Browse files
osa1Commit Queue
authored andcommitted
[infra] Add more documentation on builder names in tools/bots/test_matrix.json
Change-Id: I975ee88145ec59a4579d458f144b7de19fe5e10b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/415880 Commit-Queue: Ömer Ağacan <[email protected]> Reviewed-by: Alexander Thomas <[email protected]>
1 parent 6c81b53 commit 2337553

File tree

1 file changed

+15
-2
lines changed

1 file changed

+15
-2
lines changed

tools/bots/README.md

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,10 @@ an `options` field.
5454
"options": {
5555
"compiler": "dartk",
5656
"mode": "release",
57-
}},
57+
}
58+
},
5859
```
5960

60-
6161
### Builder Configurations
6262

6363
The builder configurations describes all test configurations a specific builder
@@ -131,6 +131,19 @@ option. Options can be runtimes (e.g. "chrome"), architectures (e.g. x64) and
131131
operating system families (e.g. win). For each valid option, additional
132132
arguments are passed to the `tools/build.py` script.
133133

134+
Parsing of builder names is done in [1]. A name like
135+
"front-end-nnbd-win-release-x64" is parsed into variables:
136+
137+
- `info`: `"front-end-nnbd"`
138+
- `os`: `"win"`
139+
- `mode`: `"release"`
140+
- `arch`: `"x64"`
141+
142+
Valid values of these variables are defined in [2].
143+
144+
[1]: https://dart.googlesource.com/recipes/+/535cf6e8ef1e11449acfc13c7bc61a530b765bd8/recipe_modules/dart/api.py#886
145+
[2]: https://dart.googlesource.com/recipes/+/535cf6e8ef1e11449acfc13c7bc61a530b765bd8/recipe_modules/dart/__init__.py
146+
134147
### Adding a new builder
135148
To add a builder:
136149

0 commit comments

Comments
 (0)