search

bytecodealliance / wit-bindgen

A language binding generator for WebAssembly interface types
Apache License 2.0
1.03k stars 195 forks source link

temporarily remove moonbit from CI language matrix #1068

Closed pchickey closed 1 month ago

pchickey commented 1 month ago

The moonbit CI job seems to be broken (https://github.com/bytecodealliance/wit-bindgen/actions/runs/11407779008/job/31744559397) due to some change to the compiler. This PR disables the CI jobs temporarily. Maybe when we re-enable it, we should pin a compiler version?

pchickey commented 1 month ago

Agreed thats preferable, but I couldn't figure out the right incantation to do that on a matrix for just one column, some googling indicates its not supported https://github.com/actions/runner/issues/2347

alexcrichton commented 1 month ago

cc @peter-jerry-ye to make sure you see this

peter-jerry-ye commented 1 month ago

@alexcrichton Thank you for letting me know. The MoonBit is still an evolving language, and the standard library is still evolving, so I expect there'll be failures from time to time.

How about I create a separate CI file that executes only on pull request (when there are changes for wit-bindgen-moonbit only) and on a weekly basis?

alexcrichton commented 1 month ago

Ah ok makes sense. Personally I find intended-to-fail CI pretty hard to wrangle so unfortunately I'm not sure of a good way to manage that. Failures on PRs look scary to contributors "not in-the-know" so I'd be hesitant to do that and a cron-based schedule often sends failure emails to the wrong person (e.g. the commit author rather than those interested). For now it may be best to leave it disabled until MoonBit has stabilized some more?