Commit 9a1d408
authored
Add pre commit to verify test status (#841)
Previously if pre-commit failed, it would cause `run-tests` to be
skipped, which would then mean that `verify-tests-status` would give the
PR the green light. That's obviously a problem, so we need to make sure
all these tests are added to the verify-tests-status check. Ideally we
could put some more logic in that if block to check if we're in a merge
queue, whether tests were intentionally skipped, etc.
---------
Signed-off-by: Peter St. John <pstjohn@nvidia.com>1 parent 6d0c0d6 commit 9a1d408
File tree
2 files changed
+5
-2
lines changed- .github/workflows
- docs/docs/models
2 files changed
+5
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
210 | 213 | | |
211 | 214 | | |
212 | 215 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
150 | 150 | | |
151 | 151 | | |
152 | 152 | | |
153 | | - | |
| 153 | + | |
154 | 154 | | |
155 | 155 | | |
156 | 156 | | |
157 | | - | |
| 157 | + | |
158 | 158 | | |
159 | 159 | | |
160 | 160 | | |
| |||
0 commit comments