Feature Request: Add support for the pre-commit framework about cloudformation-guard HOT 7 CLOSED

Looks like pre-commit natively supports rust as a buildable language plugin. If this is the case, this should be straightforward to offer.

from cloudformation-guard.

This is very reasonable ask. We're looking at integrations right now. We'll pull this into that discussion.

from cloudformation-guard.

There are still some hurdles (at least the last time I tried):

• You need to have rust installed (not much to do about that, and you can install cfn-guard yourself and use the system "language")
• pre-commit passes all matching files on one invocation (or at least does so by default). I couldn't get it to invoke cfn-guard multiple times (that's why I created #62 )

from cloudformation-guard.

+1 to this issue. Pre-commits are a vital way of remediating latent vulnerabilities before they're committed.

from cloudformation-guard.

Do we have any update on this Feature request to support pre-commit hooks?

from cloudformation-guard.

if the project already have a .pre-commit-confg.yaml i see the below config works

  - repo: local
    hooks:
      - id: cfn-guard
        name: Run Cfn Guard
        language: system
        entry: "cfn-guard validate -r ./cfn-rules/cloudformation/aws/ --data ./cdk.out/"
        pass_filenames: false
       

from cloudformation-guard.

Hi @kgrvamsi we do not have any updates here yet.

from cloudformation-guard.