Skip to content

docs: adding pre-commit functionality#136

Merged
amdove merged 4 commits into
mainfrom
pre-commit
Feb 23, 2026
Merged

docs: adding pre-commit functionality#136
amdove merged 4 commits into
mainfrom
pre-commit

Conversation

@amdove
Copy link
Copy Markdown
Contributor

@amdove amdove commented Feb 20, 2026

Description

  • Add pre-commit to repository. Replaces old custom post-checkout hooks that were unused as far as I can tell. Currently only runs format and lint, and tests for each area only if changes are made.

  • Updated docs and just recipes to support this, and included information on how to bypass if needed.

  • Cleaned up CONTRIBUTING vs README confusion, README is for users of ptd cli, CONTRIBUTING has additional information for developer set up.

    Hooks

    Hook Triggers On
    format All commits
    check All commits
    test-python-pulumi python-pulumi/**/*.py
    test-cmd cmd/**/*.go
    test-lib lib/**/*.go
image

Changes

  • .pre-commit-config.yaml - New file with 5 hooks
  • Justfile
    • Added install-git-hooks recipe (runs uvx pre-commit install)
    • deps now runs install-git-hooks automatically
  • CONTRIBUTING.md
    • Added Git Hooks section explaining behavior
    • Added bypass instructions (--no-verify, SKIP=)

Setup

just deps  # or: just install-git-hooks

Category of change

  • Bug fix (non-breaking change which fixes an issue)
  • Version upgrade (upgrading the version of a service or product)
  • New feature (non-breaking change which adds functionality)
  • Build: a code change that affects the build system or external dependencies
  • Performance: a code change that improves performance
  • Refactor: a code change that neither fixes a bug nor adds a feature
  • Documentation: documentation changes
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@amdove amdove changed the title test: adding pre-commit functionality docs: adding pre-commit functionality Feb 20, 2026
@amdove amdove marked this pull request as ready for review February 23, 2026 16:10
@amdove amdove requested a review from a team as a code owner February 23, 2026 16:10
@amdove amdove added this pull request to the merge queue Feb 23, 2026
Merged via the queue into main with commit 49269c8 Feb 23, 2026
1 check passed
@amdove amdove deleted the pre-commit branch February 23, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants