feat: simplify args inputs for terraform_docs

[1oglop1]

This should fix #154.
I removed getopt and instead simply passed args array in the same order as specified via .pre-commit-config.yaml file.

example usage:

  - repo: https://github.com/antonbabenko/pre-commit-terraform
    rev: <version>
    hooks:
      - id: terraform_docs
        verbose: true
        name: Terraform docs
        description: Inserts input and output documentation into README.md (using terraform-docs).
        require_serial: true
        entry: _ignore/pre-commit-terraform/terraform_docs.sh
        language: script
        files: (\.tf)$
        exclude: \.terraform\/.*$
        args:
          - --header-from=doc.tf
          - --hide-all
          - md

[1oglop1]

@antonbabenko Take this as an initial proposal, there is some additional cleanup required but I'll put more effort after discussion.

But I think that ideal solution which could resolve more issues regarding the version would be to discard bash and write some Go code with the same functionality and PR it to the original repository, however, that might be more work.

[1oglop1]

nudge @antonbabenko could you please provide your opinion on this? PR received a few reactions from which I assume it's a wanted feature by more people.

Thank you.

[antonbabenko]

Hi @1oglop1 ! There is simply not enough time for me to review all the projects I am involved in. I assigned @robinbowes to take a look if he has some time.

[GenPage]

@robinbowes Any update on your review?

[1oglop1]

@Vlaaaaaaad I noticed you liked my PR, and since @antonbabenko takes preference on issues/PRs from sponsors it may help if you participate in the discussion to give @antonbabenko incentive to look and comment on this change

[maulik-vimond]

Can this be merged? Out of the box terraform-docs does not work right now unless we downgrade terraform-docs.

And to make it worse homebrew does not have older versions available.

[robinbowes]

does not work right now

What doesn't work? It's working for me.

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v3.4.0
  hooks:
    - id: check-yaml
    - id: trailing-whitespace
    - id: end-of-file-fixer
- repo: https://github.com/antonbabenko/pre-commit-terraform
  rev: v1.47.0
  hooks:
    - id: terraform_validate
      args:
        - '--envs=AWS_DEFAULT_REGION=us-east-1'
    - id: terraform_fmt
    - id: terraform_docs

 $ terraform-docs --version
terraform-docs version v0.11.2 darwin/amd64 BuildDate: 2021-02-23T00:25:32+0000

[maulik-vimond]

What doesn't work? It's working for me.

How is your terraform code structured? Mine is in a sub-folder. Please see the issue #181.

Actually this PR will not help with my issue since the path is hardcoded in terraform_docs.sh. I was hoping initially that by using the fix from the PR I will be able to pass in the target module directory.

[maulik-vimond]

To provide backward compatibility shouldn't $args have md default somewhere?

[ezio-ballarin]

I also noticed I can't pass args to the terraform-docs hook. here's my config:

repos:
- repo: git://github.com/antonbabenko/pre-commit-terraform
  rev: v1.50.0 
  hooks:
    - id: terraform_fmt
    - id: terraform_docs
      args: 
        - '--args=--sort-by=required --hide=providers'

~$ pre-commit -V
pre-commit 2.13.0
~$ terraform-docs version
terraform-docs version v0.13.0 linux/amd64 BuildDate: 2021-05-27T15:35:35-07:00

[7adityaraj]

We should also hide this print or should echo when verbose is enabled.

[MaxymVlasov]

Too outdated