search

exercism / v2-configlet

Tool to assist in managing Exercism language tracks.
MIT License
16 stars 23 forks source link

WIP: starting to restructure fixtures and modify track config struct #187

Open ekingery opened 4 years ago

ekingery commented 4 years ago

@ErikSchierboom I wanted to get this PR going early so that you could validate the V3 changes to the fixtures/lint/valid-track/config.json before I spend too much time writing code against them.

ekingery commented 4 years ago

The code changes for linting should be fairly straightforward, but I'm concerned about the scope of the repercussions to the rest of the features (fmt, tree, generate). This is not yet an informed opinion, but my gut says that it might be faster and easier to essentially rewrite these tools using json-schema (or maybe joi) as suggested in #152. For your consideration, @ErikSchierboom.

ErikSchierboom commented 4 years ago

The code changes for linting should be fairly straightforward, but I'm concerned about the scope of the repercussions to the rest of the features (fmt, tree, generate). This is not yet an informed opinion, but my gut says that it might be faster and easier to essentially rewrite these tools using json-schema (or maybe joi) as suggested in #152. For your consideration, @ErikSchierboom.

I like the idea of using JSON schema to validate the track config.