-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
build: Add initial Claude Code context #93549
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
||
## Key Commands | ||
|
||
### Development Setup |
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.
I see sentry devserver
is mentioned in the Debugging section, should that be moved up here?
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.
@trevor-e unlikely to matter - this is consumed by the LLM
CLAUDE.md
Outdated
flake8 | ||
black --check | ||
isort --check |
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.
i've got this in my context:
## Linting and Code Quality
- Run pre-commit on specific files: `pre-commit run --files src/sentry/path/to/file.py`
that's usually better as running flake8 before black will make claude grind through formatting issues manually.
9d3d86d
to
5c3163e
Compare
5c3163e
to
49616aa
Compare
Think this is an ok starting point. Lets not expect perfection. If you are using Claude Code and you find some common mistakes, feel free to publish improvements to this file. |
Baseline conext for Claude Code. Avoids the cost of anyone else doing this, and will be used as we experiment with Claude Code in this repo.
Baseline conext for Claude Code. Avoids the cost of anyone else doing this, and will be used as we experiment with Claude Code in this repo.