search

exercism / pharo-smalltalk

Exercism exercises in Pharo.
https://exercism.org/tracks/pharo-smalltalk
MIT License
34 stars 28 forks source link

Update contributing guidelines #557

Closed Bajger closed 1 year ago

Bajger commented 1 year ago
Bajger commented 1 year ago

@glennj Please look, if changes in Contributing guidelines makes sense to you. I hope it is more clear, I added some necessary details, which weren't obvious to me, when doing changes.

Bajger commented 1 year ago

@ErikSchierboom Hello Erik! I'm bit unsure about using bin/configlet sync command. Is it needed to produce file artefacts when creating new from problem specification? I didn't mention this in contributing guidelines so far. Thanks for reply!

ErikSchierboom commented 1 year ago

Is it needed to produce file artefacts when creating new from problem specification? I didn't mention this in contributing guidelines so far. Thanks for reply!

Well, it is not needed per se (you could manually copy the data), but I would recommend using configlet sync.

Bajger commented 1 year ago

I've updated the instructions with up-to-date information. So potential contributors can use it in future. I'm aware some of the old code, when I wrote instructions. (e.g. configlet generate is not needed for practise exercise generation - so it needs to be removed, instead of it a sync command should be invoked, when exercise is generated from Pharo image).