search

exercism / coffeescript

Exercism exercises in CoffeeScript.
https://exercism.org/tracks/coffeescript
MIT License
19 stars 50 forks source link

[v3] Move Practice Exercise example files to .meta directory #140

Closed ErikSchierboom closed 2 years ago

ErikSchierboom commented 3 years ago

The example implementation files for Practice Exercises should be located in the .meta directory. Besides being consistent with Concept Exercises, it also has the added benefit that no extra configuration is needed to prevent the example file from being downloaded to the student (as files in the .meta directory are never downloaded).

See the spec.

This PR attempts to detect these example files and move them to the exercise's .meta directory.

Note that this change will likely break your CI, so feel free to either merge this and fix the build later, or push a fix to this PR.

Tracking

https://github.com/exercism/v3-launch/issues/18

angelikatyborska commented 2 years ago

Exercise configs have been added since this PR was opened so those would need to get updated with the new example file locations. I made a PR to your branch with that change (+ merge main) so that you don't have to, you can just merge: https://github.com/ErikSchierboom/coffeescript/pull/3

Edit: on second thought, merging another PR into this branch might be a terrible idea if we're using squash merging if that PR contains a merge main commit.. maybe don't merge, maybe merge main to your branch first and then cherry-pick my last commit? 🙈 That sounds like exactly the same amount of work as just doing the change yourself, heh. I tried to help but didn't think this through :D

ErikSchierboom commented 2 years ago

@angelikatyborska Actually, I just had to re-run my script :)