statik icon indicating copy to clipboard operation
statik copied to clipboard

Content field from _all.yml files are now loaded.

Open jayvdb opened this issue 6 years ago • 2 comments

This is 44c12a9b from #67 , fixed up to avoid test failures.

I've not attempt to built test cases yet, as I suspect a bit more code is needed.

No doubt the commit message is correct that it allows _all.yml entries to contain a Content field.

But then shouldnt individual entry .yaml files also be able to have a Content field. It looks like the code doesnt allow for that yet.

jayvdb avatar Nov 17 '18 16:11 jayvdb

Individual files have an implicit Content field, which is in fact the content of the file (the part of the file after the header).

PaulPichaureau avatar Nov 17 '18 16:11 PaulPichaureau

Hiya @PaulPichaureau , I was referring to individual .yaml files - I suspect they are able to have a Content field, from only a quick look at the code. If that is true, then that needs to be added, but I suspect that is a really easy fix. Last night I wsa only wanting to split PR 67 into two chunks which could be separately reviewed, enhanced and completed.

jayvdb avatar Nov 18 '18 00:11 jayvdb