Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

Add Terraform support #307

Merged
merged 1 commit into from
Feb 28, 2022
Merged

Add Terraform support #307

merged 1 commit into from
Feb 28, 2022

Conversation

pbar1
Copy link
Contributor

@pbar1 pbar1 commented Feb 23, 2022

Description

Added support for Terraform as a language to be enabled via doom_modules.lua. Terraform is a little weird, as it's Tree-sitter parser is simply hcl (the config language Terraform is written in) - hence the extra logic here.

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

@pbar1
Copy link
Contributor Author

pbar1 commented Feb 25, 2022

Hmm - the linter is showing a failure due to a warning, though it's not related to the changed files:

Checking lua/doom/extras/autocmds/init.lua        1 warning

    lua/doom/extras/autocmds/init.lua:169:9: unused variable show_diagnostics_function
Error: unused variable show_diagnostics_function

@NTBBloodbath
Copy link
Collaborator

Hey, sorry for the delay in my response.

Thank you so much per adding this, I was also thinking about start learning terraform in a near future so it also comes handy for me haha.

Regarding to linter we can just ignore it because it's not related to PR changes and needs to be fixed externally.

Cheers!

@NTBBloodbath NTBBloodbath added the scope: enhancement New feature or request label Feb 28, 2022
@NTBBloodbath NTBBloodbath merged commit ab3a03c into doom-neovim:main Feb 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
scope: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants