Skip to content

Latest commit

 

History

History
138 lines (104 loc) · 5.7 KB

CONTRIBUTING.md

File metadata and controls

138 lines (104 loc) · 5.7 KB

Notes for contributors

  1. Python hooks are supported now too. All you have to do is:
    1. add a line to the console_scripts array in entry_points in setup.py
    2. Put your python script in the pre_commit_hooks folder

Enjoy the clean, valid, and documented code!

Run and debug hooks locally

pre-commit try-repo {-a} /path/to/local/pre-commit-terraform/repo {hook_name}

I.e.

pre-commit try-repo /mnt/c/Users/tf/pre-commit-terraform terraform_fmt # Run only `terraform_fmt` check
pre-commit try-repo -a ~/pre-commit-terraform # run all existing checks from repo

Running pre-commit with try-repo ignores all arguments specified in .pre-commit-config.yaml.

If you need to test hook with arguments, follow pre-commit doc to test hooks.

For example, to test that the terraform_fmt hook works fine with arguments:

/tmp/pre-commit-terraform/terraform_fmt.sh --args=-diff --args=-write=false test-dir/main.tf test-dir/vars.tf

Run hook performance test

To check is your improvement not violate performance, we have dummy execution time tests.

Script accept next options:

# Name Example value Description
1 TEST_NUM 200 How many times need repeat test
2 TEST_COMMAND 'pre-commit try-repo -a /tmp/159/pre-commit-terraform terraform_tfsec' Valid pre-commit command
3 TEST_DIR '/tmp/infrastructure' Dir on what you run tests.
4 TEST_DESCRIPTION '`terraform_tfsec` PR #123:' Text that you'd like to see in result
5 RAW_TEST_
RESULTS_FILE_NAME		 terraform_tfsec_pr123 (Temporary) File where all test data will be stored.

Run via BASH

# Install deps
sudo apt install -y datamash
# Run tests
./hooks_performance_test.sh 200 'pre-commit try-repo -a /tmp/159/pre-commit-terraform terraform_tfsec' '/tmp/infrastructure' '`terraform_tfsec` v1.51.0:' 'terraform_tfsec_pr159'

Run via Docker

# Build `pre-commit` image
docker build -t pre-commit --build-arg INSTALL_ALL=true .
# Build test image
docker build -t pre-commit-tests tests/
# Run
TEST_NUM=1
TEST_DIR='/tmp/infrastructure'
PRE_COMMIT_DIR="$(pwd)"
TEST_COMMAND='pre-commit try-repo -a /pct terraform_tfsec'
TEST_DESCRIPTION='`terraform_tfsec` v1.51.0:'
RAW_TEST_RESULTS_FILE_NAME='terraform_tfsec_pr159'

docker run -v "$PRE_COMMIT_DIR:/pct:rw" -v "$TEST_DIR:/lint:ro" pre-commit-tests \
    $TEST_NUM "$TEST_COMMAND" '/lint' "$RAW_TEST_RESULTS_FILE_NAME" "$RAW_TEST_RESULTS_FILE_NAME"

Check results

Results will be located at ./test/results dir.

Cleanup

sudo rm -rf tests/results

Add new hook

You can use this PR as an example.

Before write code

  1. Try to figure out future hook usage.
  2. Confirm the concept with Anton Babenko.

Prepare basic documentation

  1. Identify and describe dependencies in Install dependencies and Available Hooks sections

Add code

  1. Based on prev. block, add hook dependencies installation to Dockerfile.
    Check that works:
    • docker build -t pre-commit --build-arg INSTALL_ALL=true .
    • docker build -t pre-commit --build-arg <NEW_HOOK>_VERSION=latest .
    • docker build -t pre-commit --build-arg <NEW_HOOK>_VERSION=<1.2.3> .
  2. Add new hook to .pre-commit-hooks.yaml
  3. Create hook file. Don't forget to make it executable via chmod +x /path/to/hook/file.
  4. Test hook. How to do it is described in Run and debug hooks locally section.
  5. Test hook one more time.

    1. Push commit with hook file to GitHub

    2. Grab SHA hash of the commit

    3. Test hook using .pre-commit-config.yaml:

      repos:
- repo: https://github.com/antonbabenko/pre-commit-terraform # Your repo
rev: 3d76da3885e6a33d59527eff3a57d246dfb66620 # Your commit SHA
hooks:
  - id: terraform_docs # New hook name
    args:
      - --args=--config=.terraform-docs.yml # Some args that you'd like to test

Finish with the documentation

  1. Add hook description to Available Hooks.
  2. Create and populate a new hook section in Hooks usage notes and examples.