- Notifications
You must be signed in to change notification settings - Fork22
Claude.md#619
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Claude.md#619
Uh oh!
There was an error while loading.Please reload this page.
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
| # Bash commands | ||
| - `ruff check --fix --exit-non-zero-on-fix --config=pyproject.toml .`: Run the linter | ||
| - `ruff format .`: Run the formatter | ||
| - `mypy --non-interactive --config-file pyproject.toml @mypy_allowlist.txt`: mypy typechecker | ||
| # Running tests | ||
| - Run tests with `pytest tests/` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
these will all fail because we're using uv in order to run these commands, maybe using hooks is a better approachhttps://docs.anthropic.com/en/docs/claude-code/hooks-guide
Uh oh!
There was an error while loading.Please reload this page.
PR Type
Documentation
Description
Add
Claude.mdwith linting and formatting commandsInclude
ruffandmypyinstructions for code checksAdd placeholder Code style section
Add workflow tips: typecheck and focused tests
File Walkthrough
Claude.md
Create Claude.md linting and workflow guideClaude.md
ruff check --fixandruff formatcommandsmypytypechecking instructions