Disallow whitespace in some multi-fragment constructs.

camdencheek / tree-sitter-dockerfile

A tree-sitter grammar for Dockerfile

MIT License

71 stars 20 forks source link

Closed mjambon closed 2 years ago

mjambon commented 2 years ago

This is more correct (#19) and ends up solving (#18).

camdencheek commented 2 years ago

This looks great! Thank you for the contribution and all the tests.

It's unfortunate that it's so awkward to disallow spaces in a construct, but I think this is still preferable to getting rid of the extras.

mjambon commented 2 years ago

Yes. I briefly considered removing the extras and figured it would be worse.