search

nodejs / uvwasi

WASI syscall API built atop libuv
MIT License
226 stars 50 forks source link

add debug log CI and consolidate jobs #132

Closed cjihrig closed 4 years ago

cjihrig commented 4 years ago

The first commit adds a CI run to test the UVWASI_DEBUG_LOG feature.

The standard, debug log, and ASAN jobs all seem to work fine across Linux, macOS, and Windows. The second commit combines them into a single job matrix.

sbc100 commented 4 years ago

Seems reasonable. Is it worth running all config on all operating systems? Would it be enough to run the full matrix on just linux and then the standard build on all OSes?

(I guess they all run very quick so its fine for now).

cjihrig commented 4 years ago

Would it be enough to run the full matrix on just linux and then the standard build on all OSes?

I'd like to do that + run ASAN on Windows. I'll experiment with some filtering later today.

cjihrig commented 4 years ago

This is updated to only run the tests exactly where they should.