jm / toml

Parse TOML. Like a bawss.
MIT License
151 stars 38 forks source link

make toml work with parslet >= 1.7 #44

Closed boutil closed 6 years ago

boutil commented 8 years ago

This change makes toml work with newer parslet. As the order of insertion of rules have changed in parslet to give newer rules higher priority, rules have been exchanged in transform.rb.

This closes #43.

jm commented 8 years ago

Hm something is weird with CI on this for some reason. I'll look into it, and if I can figure it out, I'll merge it (heck, from the looks of it, it's unrelated so I'll merge it anyhow but I'd like to get it figure it beforehand if possible...).

aaronlippold commented 6 years ago

Hi, any progress on this, this is causing an issue with some of my work with InSpec and Parslet ( which is now up to v1.8 ) so would love to see this integrated and then the overall requirement to it be relaxed to greater than or equal to 1.7.1 if that is the lowest testing base if that makes sense rather than to a hard static version.

jm commented 6 years ago

Just merged another branch that fixed it for newer Parslet versions! So very sorry I let this branch decay, but fortunately someone else PR'd another up to date one recently. 😄