Skip to content

Commit 30e61d7

Browse files
fix(update-expected): offer more general language and aliases
1 parent 0c32d99 commit 30e61d7

1 file changed

Lines changed: 9 additions & 7 deletions

File tree

moz-webgpu-cts/src/main.rs

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -63,20 +63,20 @@ struct Cli {
6363
#[derive(Debug, Parser)]
6464
enum Subcommand {
6565
/// Adjust expected test outcomes in metadata, optionally using `wptreport.json` reports from
66-
/// CI runs covering Firefox's implementation of WebGPU.
66+
/// CI runs covering your browser's implementation of WebGPU.
6767
///
68-
/// As Firefox's behavior changes, one generally expects CTS test outcomes to change. When you
69-
/// are testing your own changes in CI, you can use this subcommand to update expected outcomes
70-
/// automatically with the following steps:
68+
/// As your browser's behavior changes, one generally expects CTS test outcomes to change. When
69+
/// you are testing your own changes in CI, you can use this subcommand to update expected
70+
/// outcomes automatically with the following steps:
7171
///
72-
/// 1. Run `moz-webgpu-cts update-expected --preset=new-fx …` against the first complete set of
73-
/// reports you gather from CI with your new Firefox build. This will adjust for new
72+
/// 1. Run `moz-webgpu-cts update-expected --preset=new-build …` against the first complete set
73+
/// of reports you gather from CI with your new browser build. This will adjust for new
7474
/// permanent outcomes, and may capture some (but not all) intermittent outcomes.
7575
///
7676
/// 2. There may still exist intermittent issues that you do not discover in CI run(s) from the
7777
/// previous step. As you discover them in further CI runs on the same build of Firefox,
7878
/// adjust expected outcomes to match by running `moz-webgpu-cts update-expected
79-
/// --preset=same-fx …` against the runs' new reports. Repeat as necessary.
79+
/// --preset=same-build …` against the runs' new reports. Repeat as necessary.
8080
///
8181
/// With both steps, you may delete the local copies of these reports after being processed
8282
/// with `update-expected`. You should not need to re-process them unless you have made an
@@ -110,9 +110,11 @@ enum Subcommand {
110110
enum ReportProcessingPreset {
111111
/// alias: `new-fx`
112112
#[value(alias("new-fx"))]
113+
#[value(alias("new-build"))]
113114
ResetContradictory,
114115
/// alias: `same-fx`
115116
#[value(alias("same-fx"))]
117+
#[value(alias("same-build"))]
116118
Merge,
117119
ResetAll,
118120
}

0 commit comments

Comments
 (0)