odin-lang / Odin

Odin Programming Language
https://odin-lang.org
BSD 3-Clause "New" or "Revised" License
6.66k stars 584 forks source link

Utilise `odin test` instead of scripts to run tests #3698

Closed laytan closed 4 months ago

laytan commented 4 months ago

Started as wanting to remove all the Makefile and build.bat scripts involved in running tests, but also found some other issues I took care of.

  1. remove most Makefile and build.bat files for running tests, utilise the odin test -all-packages to collect and run all the tests
  2. Put ubuntu, macos intel and macos arm on the same ci steps using a matrix, so we don't have slight differences anymore
  3. Instead of giving every step an individual timeout, give the entire run a timeout, is clearer, and more robust because we had some steps without a timeout
  4. move to actions/checkout@v4 everywhere, I noticed the netbsd action was using it and good to be consistent
  5. Fix the A union with #no_nil must have at least 2 variants error message, it was trying to put the error on the align ast node, which might be nil
  6. Fix not printing Error: when the terminal doesn't support colours