exercism / d

Exercism exercises in D.
https://exercism.org/tracks/d
MIT License
19 stars 30 forks source link

Add knapsack exercise #253

Closed keiravillekode closed 2 months ago

keiravillekode commented 2 months ago

I tried to submit my knapsack solution through the Exercism website, and received compile errors in the tests:

 Error: found `:` when expecting `,`

I can change the tests to replace

Item(weight: 120, value: 240)

with

Item(120, 240)

Less urgently, we might consider updating the D compiler used by Exercism.

BNAndras commented 2 months ago

The CI targets the latest version which isn’t good. We should target specific versions or use the test runner image to test the exercises.

BNAndras commented 2 months ago

I added PRs to pin the CI's dmd version for this repo and to also upgrade the Docker image to dmd 2.109.0.