Add CLAUDE.md with project structure and commands#4233
Add CLAUDE.md with project structure and commands#4233Kludex wants to merge 3 commits intoopen-telemetry:mainfrom
Conversation
aabmass
left a comment
aabmass
left a comment
There was a problem hiding this comment.
Looks like a good starting point, thanks!
There was a problem hiding this comment.
Can we rename to AGENTS.md or symlink to it?
There was a problem hiding this comment.
I don't think Claude checks AGENTS.md, or there's a setting to change that. I've added a symlink.
There was a problem hiding this comment.
I think the main content should be in AGENTS.md then add @AGENTS.md to CLAUDE.md - see here.
This makes sure other agents (co-pilot, cursor, etc) follow the guidance, not just Claude.
There was a problem hiding this comment.
I think the main content should be in AGENTS.md then add @AGENTS.md to CLAUDE.md - see here.
This makes sure other agents (co-pilot, cursor, etc) follow the guidance, not just Claude.
| uv run tox -e typecheck | ||
| ``` | ||
|
|
||
| ## Guidelines |
There was a problem hiding this comment.
There are some other nice bits in the AGENTS file the Collector added, especially the Assisted-by commit annotation.
There was a problem hiding this comment.
Yes, please. I think we also have started to receive PRs from bot / people throwing a bot at good-first-issue labeled issues.
tammy-baylis-swi
moved this from Approved PRs
to Ready for review
in @xrmx's Python PR digest
| - `exporter/` - custom exporters | ||
| - `propagator/` - context propagators | ||
|
|
||
| Each package lives under `src/opentelemetry/instrumentation/{name}/` with its own `pyproject.toml` and `tests/`. |
There was a problem hiding this comment.
src/opentelemetry/instrumentation is only for instrumentations
| uv run tox -e typecheck | ||
| ``` | ||
|
|
||
| ## Guidelines |
There was a problem hiding this comment.
Yes, please. I think we also have started to receive PRs from bot / people throwing a bot at good-first-issue labeled issues.
4 participants
Based on discussion from last week's meeting.
It's minimal.
You can't prevent people from spamming your repo with thousands of AI based PRs, but you can guide their AI to do the right thing.