ctags icon indicating copy to clipboard operation
ctags copied to clipboard

Terraform (HCL) (*.tf): new parser

Open antonysouthworth-halter opened this issue 3 years ago • 7 comments

Add support for Terraform (HCL) files.

Related:

  • https://github.com/preservim/tagbar/issues/757

TODO

  • [ ] add unit tests
  • [ ] update documentation
    • [ ] manpages
    • [ ] codebase

antonysouthworth-halter avatar Apr 11 '21 22:04 antonysouthworth-halter

Have opened this as a quick thing, please advise if there are other things I need to do before merging (I read through https://docs.ctags.io/en/latest/optlib.html but may have missed some things).

antonysouthworth-halter avatar Apr 11 '21 22:04 antonysouthworth-halter

@masatake thank you for fast and thorough review 😄, I will address comments as/when I can.

@juliosueiras apologies, perhaps I should have talked to you first? 😅

antonysouthworth-halter avatar Apr 11 '21 23:04 antonysouthworth-halter

@antonysouthworth-halter no, thats fine

juliosueiras avatar Apr 11 '21 23:04 juliosueiras

I was able to add locals parsing if this PR still has any traction.

tonglil avatar Aug 10 '21 21:08 tonglil

@tonglil, feel free to make a new pull request that includes the changes of this pull request. (It means you take over this pull request.)

masatake avatar Aug 10 '21 21:08 masatake

The biggest reason I didn't merge this is there is no test case. @tonglil do you have an interest in polishing this pull request?

masatake avatar Aug 10 '21 22:08 masatake

Sorry, I will not be able to contribute to this PR.

tonglil avatar Jan 26 '22 21:01 tonglil

@masatake I'd like to take over and finish this PR

Haggus avatar Mar 29 '23 16:03 Haggus

@Haggus You are welcome. optlib/terraform.c is automatically generated by misc/optlib2c. So what you have to edit is optlib/terraform.ctags. Test cases are needed.

masatake avatar Mar 29 '23 18:03 masatake

thanks @Haggus , unfortunately I never got time to come back to this. @masatake should we close this PR in favour of #3683 ?

antonysouthworth-halter avatar Mar 30 '23 20:03 antonysouthworth-halter

I will close this via #3684.

masatake avatar Mar 31 '23 03:03 masatake

@antonysouthworth-halter, thank you.

masatake avatar Apr 01 '23 19:04 masatake