Skip to content

Add AGENTS.md contributor guide + CLAUDE.md pointer - #268

Merged
kali merged 1 commit into
mainfrom
docs/claude-md
Aug 16, 2026
Merged

Add AGENTS.md contributor guide + CLAUDE.md pointer#268
kali merged 1 commit into
mainfrom
docs/claude-md

Conversation

@kali

@kali kali commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Overview code map + Contributing rules (code style, commit messages, inline/doc comments, PR bot/human interaction), adapted from tract. CLAUDE.md just points at AGENTS.md.

@kali
kali requested a review from fredszaq August 16, 2026 17:06
@kali

kali commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator Author

Any opinion ? They will only read it when they feel like it anyway...

Comment thread AGENTS.md Outdated
## Contributing

### Before you commit
- Format with `cargo fmt --all`. The repo pins `rust-toolchain` (1.85.0), so bare

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we're actually testing against 1.88 and don't pin the toolchain, we probably should

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well there is a rust-toolchain at dinghy top level...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

arf didn't see it, complicated week-end :P. probably a better idea not to have the 1.85.0 stated here, just saying the rust-toolchain file exist, should be less prone to the information becoming stale in the future

Comment thread CLAUDE.md Outdated
@@ -0,0 +1,6 @@
# CLAUDE.md

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this one just be an symbolic link to the agents.md file ? That way you're sure claude will have the content of it in its context. pointing it like that and you may have agents not bothering to read it (I have already seen this)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm always worried about symlinks in git. what happens on nfs ?

@fredszaq fredszaq Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On NFS pretty sure no problems, on NTFS (windows) didn't try for a long time to be honnest.

From what i see there seems to be some options now if you enable developer mode in windows, set git config --global core.symlinks true before cloning the repo. looks like if you don't to that you just have a file with the target of the symlink in it. Won't prevent building dinghy and should be quite visible if someone makes a PR from an misconfigured windows pc

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, i meant ntfs. all right. let's give it a try.

Overview code map + Contributing rules (code style, commit messages,
inline/doc comments, PR bot/human interaction), adapted from tract.
CLAUDE.md is a symlink to AGENTS.md so the content is always in context.
@kali
kali merged commit 981bbb1 into main Aug 16, 2026
10 checks passed
@kali
kali deleted the docs/claude-md branch August 16, 2026 18:04
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