rust-bitcoin / rust-bitcoin / 8877622800

83%

push

github

Merge rust-bitcoin/rust-bitcoin#2633: CI: Re-write `run_task.sh`

<a class=hub.com/rust-bitcoin/rust-bitcoin/commit/<a class="double-link" href="https://git"><a class=hub.com/rust-bitcoin/rust-bitcoin/commit/26b9782d8bcd2232249219bb0ccfd5fb391ae9c0">26b9782d8 CI: Re-write run_task.sh (Tobin C. Harding)

Pull request description:

  Recently we re-wrote CI to increase VM level parallelism, in hindsite this has proved to be not that great because:

  - It resulted in approx 180 jobs
  - We are on free tier so only get 20 jobs (VMs) at a time so its slow to run
  - The UI is annoying to dig through the long job list to find failures

  Have another go at organising the jobs with the main aim of shortening total run time and making it easier to quickly see fails.

  Re-write the `run_task.sh` script, notable moving manifest handling to the workflow. Also don't bother testing with beta toolchain.

  ### Note on review

  The diff is hard to read for `rust.yml`, I tried splitting out a bunch of separate patches but it resulted in the same thing (because there are so many identical lines in the yaml file). I suggest just looking at the yaml file and not the diff.

ACKs for top commit:
  apoelstra:
    ACK 26b9782d8bcd2232249219bb0ccfd5fb391ae9c0
  sanket1729:
    ACK 26b9782d8bcd2232249219bb0ccfd5fb391ae9c0.

Tree-SHA512: 1b0a0bab5c31a1b0d34a950d570ab77ae0e

19195 of 23090 relevant lines covered (83.13%)

16775.73 hits per line