search

exercism / racket-test-runner

GNU Affero General Public License v3.0
1 stars 7 forks source link

Upgrade runner to use Racket 8.11 CS #68

Closed BNAndras closed 10 months ago

BNAndras commented 11 months ago

Racket comes out every three months so the latest version came out four days ago.

BNAndras commented 11 months ago

Right, I forgot I didn't tweak the test runner script yet to sanitize file paths and those contain the Racket version number. Therefore, some of the expected results still contain a reference 8.10.

I'll take care of that this weekend.

BNAndras commented 11 months ago

I believe it's also time to switch from the Racket BC ('Before Chez') Docker image to a Racket CS ('Chez Scheme`) image. Racket CS has been the default implementation available since 8.0 was released two years ago. Judging from https://blog.racket-lang.org/2021/01/racket-status.html, the test runner should hopefully be faster. A student recently commented that list-ops was difficult to solve because there were some expensive tests being run.

ErikSchierboom commented 11 months ago

CI is failing. I think the expected values need to be updated.

BNAndras commented 11 months ago

Yeah, I'll take care of both issues.

BNAndras commented 10 months ago

As a note to myself, I wanted to add the math library to the Docker image. We downgraded the base image from the full Racket distribution to the minimal Racket distribution. That saved half a gig, but re-adding this library is non-trivial since it has a lot of nested dependencies. I'll keep the test runner using the minimal Racket distribution and look into if it'll clash with the syllabus I'll be working on next year.