Repository navigation
Commit 5795aaa
Run npm verbosely when debug mode is enabled (#340)
* Run npm verbosely when debug mode is enabled
When an npm command hangs or misbehaves during preparation, the runner
shows nothing that explains where it is stuck. Append --loglevel=silly
to the npm install and build commands when WPT_DEBUG is set, matching
how debug mode already adds verbosity to the rsync operations. Default
behavior is unchanged.
Based on the approach in #221 by @rollecode, updated to gate the
verbosity behind WPT_DEBUG as suggested in the review discussion.
* Move the npm verbosity setup above the checkout comment block
Keeps the existing block attached to the perform_operations() call it describes and uses the file's multi-line comment style, as suggested in review.
---------
Co-authored-by: Amy Kamala <39748136+kittenkamala@users.noreply.github.com>1 parent d974e38 commit 5795aaa
3 files changed
Lines changed: 12 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
63 | | - | |
| 63 | + | |
| 64 | + | |
64 | 65 | | |
65 | 66 | | |
66 | 67 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
241 | 241 | | |
242 | 242 | | |
243 | 243 | | |
244 | | - | |
| 244 | + | |
245 | 245 | | |
246 | 246 | | |
247 | 247 | | |
| |||
395 | 395 | | |
396 | 396 | | |
397 | 397 | | |
398 | | - | |
| 398 | + | |
399 | 399 | | |
400 | 400 | | |
401 | 401 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
79 | 86 | | |
80 | 87 | | |
81 | 88 | | |
| |||
96 | 103 | | |
97 | 104 | | |
98 | 105 | | |
99 | | - | |
| 106 | + | |
100 | 107 | | |
101 | 108 | | |
102 | 109 | | |
| |||
0 commit comments