build: skip dockit on riscv64 - #62251
Conversation
Signed-off-by: Stewart X Addison <sxa@ibm.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #62251 +/- ##
==========================================
+ Coverage 89.68% 90.05% +0.37%
==========================================
Files 676 751 +75
Lines 206495 254919 +48424
Branches 39537 48124 +8587
==========================================
+ Hits 185204 229580 +44376
- Misses 13435 16495 +3060
- Partials 7856 8844 +988 🚀 New features to boost your workflow:
|
|
I think we can fix it (at least the |
|
|
Probably fixed with the updae in #62512 - running a test on my K1 at https://ci.nodejs.org/job/sxa-rvnodetest/399/ |
|
Noting that I hit this again today with a branch that had the doc-kit 1.4.1 added in https://github.com/nodejs/node/pull/64010/changes so this may still be required. Ref: https://ci.nodejs.org/job/sxa-node-test-commit-rvlinux/14/nodes=riscv64 |
Signed-off-by: Stewart X Addison <sxa@ibm.com>
|
Apparently the variable name in this PR didn't work for the check - changed from |
|
Noting that the request-ci triggers have been playing up. Ref: nodejs/build#4443 |
The
doc-onlytarget gives a variety of failures on different riscv64 systems. These symptoms are described in nodejs/build#4099 (comment) in the sectionAdditional info on the Wasm doc-only crashThis removes the doc-only target from being run by default as part of
make test-cisimilar to what has been done for AIX and IBMi.