fixed unneeded optional unwrap in hello_json

zigzap / zap

blazingly fast backends in zig

MIT License

1.98k stars 71 forks source link

fixed unneeded optional unwrap in hello_json #79

Closed joeypas closed 4 months ago

joeypas commented 4 months ago

hello_json example failed to compile because of optional unwrap. Fixed by removing the ".?"

renerocksai commented 4 months ago

Thx! How the heck did that not get caught by CI...

joeypas commented 4 months ago

I was thinking the same thing! Glad I could help

renerocksai commented 4 months ago

It was my build_all script that did not propagate the exit code of individual failing builds. The script is completely unnecessary as it has long be replaced by zig build all. Fixed that in the workflow.