microadam / drone-config-changeset-conditional

The Unlicense
65 stars 17 forks source link

fix(yaml parser): safer multi-document yaml split #3

Closed Pajk closed 5 years ago

Pajk commented 5 years ago

When the document contains --- that are not supposed to separate yaml documents (eg. in comments) then this plugin prevents it to be parsed and built.

Splitting the string by \n---\n is a bit safer.

microadam commented 5 years ago

Thanks! I had meant to make this change myself, but it slipped my mind