docs (modules/vnet): Fix variable description

[Lostmaniac9]

Description

Changed id in Variables.tf from network_security_group_id to network_security_group

Motivation and Context

#232

How Has This Been Tested?

I ran the pre-commit tests to what I understand to be the standards of the Contributing document.

Types of changes

Bug fix

Checklist

• [ x ] I have updated the documentation accordingly.
• [ x ] I have read the CONTRIBUTING document.
• I have added tests to cover my changes if appropriate.
• All new and existing tests passed.

I am slightly unsure if I ran the pre-commit script correctly since I don't entirely understand what it does, but I believe it ran correctly and didn't have any significant issues to report.

[welcome-to-palo-alto-networks]

🎉 Thanks for opening this pull request! We really appreciate contributors like you! 🙌

[FoSix]

@Lostmaniac9 - Thanks for opening this PR. Unfortunately we cannot merge it in this form, just two small steps to do before we can:

1. please take a look at the Contributing.md document and make sure you run pre-commit in order to update the REDME files
2. please follow Conventional Commits when titling the PR.

[Lostmaniac9]

I am a bit confused with how to proceed with this. I have been trying to run pre-commit to update the README files but it keep failing when trying to validate terraform docs and tflint and gives an exit code of 127. Any reading I have done in the Contributing.md document hasn't been able to help me. I think I haven't installed tflint and terraform correctly but I'm not sure what I am missing.

What's more is that it looks like above my comment several checks were run for my commit that all worked out so now I am wondering if anything is wrong at all.

[migara]

@Lostmaniac9 can you please paste the error you're getting here?

[Lostmaniac9]

Hoping this image show up well because the formatting in the console isn't very good. I am pretty sure I've just installed terraform and tflint incorrectly but I haven't been able to figure out how.

[migara]

@Lostmaniac9 not sure entirely. Perhaps try to install tflint and Terraform fmt via a package manager like Chocolatey.

I will fix this for this PR

[welcome-to-palo-alto-networks]

🎉 Congrats on getting your first pull request merged! We here at Palo Alto Networks are so grateful! ❤️