Skip to content

Chore/pre commit hooks 88 - #120

Open
suhaslord wants to merge 2 commits into
nasa:developfrom
suhaslord:chore/pre-commit-hooks-88
Open

Chore/pre commit hooks 88#120
suhaslord wants to merge 2 commits into
nasa:developfrom
suhaslord:chore/pre-commit-hooks-88

Conversation

@suhaslord

Copy link
Copy Markdown
Contributor

Summary

Adds pre-commit hooks to make the local development checks match the repo’s existing CI more closely.

Changes

  • added .pre-commit-config.yaml
    • runs the critical flake8 checks before commits
    • runs mypy before commits
    • runs the full pytest suite before pushes
  • added CONTRIBUTING.md
    • basic Poetry setup
    • how to install/use pre-commit
    • commands for running the checks manually

This keeps the setup pretty lightweight while catching the same kinds of issues earlier, before they reach CI.

Closes #88

@chuckwondo
chuckwondo force-pushed the chore/pre-commit-hooks-88 branch from cf11bdb to 85b70b8 Compare August 23, 2026 21:15
@chuckwondo

Copy link
Copy Markdown
Collaborator

@suhaslord, I rebased your branch on main because I merged some bot bumps, so you'll need to pull those changes on your branch, but since I rebased it, I strongly recommend you configure your git client as follows, which will prevent git pull from failing to pull the rebased branch:

git config set --global pull.rebase true

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.

Add pre-commit hooks

2 participants