bartavelle / language-puppet

A library to work with Puppet manifests, test them and eventually replace everything ruby.
BSD 3-Clause "New" or "Revised" License
51 stars 8 forks source link

Missing parsing failure when a space follow the varname before '[' #264

Open PierreR opened 5 years ago

PierreR commented 5 years ago

While fixing #260 another tiny bug shows up. This should not parsed:

  "$os ['name']"

As shows by the following pending spec: https://github.com/bartavelle/language-puppet/blob/31e6d4fad959619c920a4077ef5b535c5b411943/tests/Parser/ExprSpec.hs#L38

bartavelle commented 5 years ago

This is a general note for all the issues : I am currently busy with other things, till the end of the year. After that, I will most certainly rewrite all the parser so that it conforms to the real thing.

PierreR commented 5 years ago

Cool. Would you mind releasing 1.4.1 ? I am probably ok on my side with the current status till the end of the year.

bartavelle commented 5 years ago

done.