search

cmur2 / language-terraform

Terraform.io support for Atom
MIT License
49 stars 32 forks source link

language-terraform breaks language-todo #52

Closed agrrh closed 3 years ago

agrrh commented 3 years ago

Sadly, I can't use TODOs in my TF code anymore.

Terraform language:

Screenshot from 2020-11-26 18-37-18

Any other language, YAML in this case:

Screenshot from 2020-11-26 18-37-30

Just to mention, toggling "Use tree sitter parsers" option makes no effect.

cmur2 commented 3 years ago

Thanks for the report @agrrh, do you know if this is a new problem or was that always like this? I'm looking into how other languages do this, happy for any pointers (out of tree).

cmur2 commented 3 years ago

Guess I found a pointer https://github.com/atom/language-yaml/pull/90 to how this could be done https://github.com/atom/language-coffee-script/commit/ad0aa553d8f91ca1c4605da0bce5bcaf66c3e2bd

cmur2 commented 3 years ago

I released v0.9.6 with a fix for your problem, please check if it works for you now :)

agrrh commented 3 years ago

It works awesome, thank you!