[DOC] - terraform is needed to run the pre-commit hooks

[pmeier]

Preliminary Checks

• This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/nebari-dev/nebari/discussions

Summary

The terraform_fmt hook needs terraform to be installed in the current environment to run. This is mentioned in the requirements for this hook. This is not mentioned in the code contribution guidelines.

Might be related to #307, since there seems to be some information missing regarding the development environment.

Steps to Resolve this Issue

Add terraform as part of the development requirements

[pavithraes]

Thanks for reporting. I don't see terraform in our previous environment-dev.yml either, but we can add it to the contributor guide to install separately.

@iameskild What do you think?

[iameskild]

Adding Terraform to the contributor guide makes sense to me :)

[pmeier]

Note that it can also be installed with conda install -c conda-forge terraform. Might be helpful for folks that use a conda env anyway so they don't have to install systemwide.

[pmeier]

I just realized this is why we have skip for pre-commit CI and run this is a separate CI workflow.