Conversation
✅ Deploy Preview for nf-core-main-site ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
mashehu
left a comment
mashehu
left a comment
There was a problem hiding this comment.
can't we just symlink this to contributing.md? All this information is there already and we don't need to remember to keep things in sync
|
You mean That file is mostly about how to add yourself to the list of contributors, and how to fork the repo and make PRs. None of that is relevant to an LLM that will already be checked out on a branch with a task to do (actually it'll make its performance actively worse). It doesn't include any information about the build architecture, testing, Astro setup etc (arguably most Human contributors don't need to know any of this, as they can just edit a markdown file and see the deployment preview). |
|
Side note: playing around with how to get Claude to read
I think this should get Claude to automatically read |
|
Ah, sorry, was thinking of the wrong repo. For the website the readme has the same instructions as the agents.md |
|
That's closer, but it still doesn't mention tests etc. and is 300+ lines instead of 30. Also it's not automatically loaded into context like this file. This isn't that much to keep updated is it? Would be a good addition to this file to say read the |
|
|
||
| Tests are **Playwright** (`npm run test --workspace sites/<name>`, or `npx playwright test <file>` within a sub-site for a single test). | ||
|
|
||
| **GitHub token:** the data-driven sub-sites (`pipelines`, `pipeline-results`, `configs`, `modules-subworkflows`, `stickers`) hit the GitHub API during the build. Add a `.env` at the repo root with a `GITHUB_TOKEN` (a personal access token with `public_repo` scope) to avoid rate limits. |
There was a problem hiding this comment.
i don't think it can add a correct github_token automatically...
There was a problem hiding this comment.
If it's an agent running locally it should be able to, using the gh CLI (gh auth token off the top of my head).
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
|
We had a CLAUDE.md file originally in sarek, before we even thought about doing an AGENTS.md file, so we have this for now: https://github.com/nf-core/sarek/blob/dev/CLAUDE.md |
3 participants
Minimal
AGENTS.md, added after asking Claude to debug phantom CORS errors. Hopefully helps sessions get going a bit faster.