Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(terraform): remove duplicate resource snippet #432

Merged
merged 1 commit into from Apr 29, 2024

Conversation

hieulw
Copy link
Contributor

@hieulw hieulw commented Apr 7, 2024

Duplicated key resource in terraform snippet cause vsnip error in my neovim recently.
Probably because this merge request #426 add new resource key but forgot remove the old one. I deleted the old one because two key doing the same purpose but the new one added by #426 is more verbose and good for readability

Duplicated key `resource` in terraform snippet cause `vsnip` error in my neovim recently
Probably because this merge request rafamadriz#426 add new resource key but forgot remove the old one.
I deleted the old one because two key doing the same purpose but the new one added by rafamadriz#426 is more verbose
Copy link

@AidenAllredOfficial AidenAllredOfficial left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the duplicate at line 6978.
Seems good to me!

Copy link
Collaborator

@OkelleyDevelopment OkelleyDevelopment left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We appreciate the PR!

@OkelleyDevelopment OkelleyDevelopment merged commit 9e55019 into rafamadriz:main Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants