docs: add configuration files to local workspace files#25619
docs: add configuration files to local workspace files#25619mickael-docker wants to merge 1 commit into
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
The changes are accurate, well-scoped, and match the existing document tone. A small formatting nit is noted inline.
| that Git hooks live inside `.git/` and do not appear in `git diff` output — | ||
| check them separately. See | ||
| CI configuration, IDE task configs, AI project configuration and settings, | ||
| `Makefile`,`package.json` scripts, and similar build files. Review changes |
There was a problem hiding this comment.
[LOW] Missing space after comma between consecutive inline code spans
The new line reads: `Makefile`,`package.json` — there is no space after the comma separating the two code spans. In rendered Markdown this produces 'Makefile','package.json' with no whitespace, making the list harder to scan. Add a space: `Makefile`, `package.json`.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
Two issues found in the added lines — one formatting error (missing space) and one content gap (Cursor omitted from the AI tool list in a security-warning context).
| that Git hooks live inside `.git/` and do not appear in `git diff` output — | ||
| check them separately. See | ||
| CI configuration, IDE task configs, AI project configuration and settings, | ||
| `Makefile`,`package.json` scripts, and similar build files. Review changes |
There was a problem hiding this comment.
[MEDIUM] Missing space after comma between Makefile and package.json code spans
The line reads `Makefile`,`package.json` scripts — there is no space after the comma between the two inline code spans. Every other comma-separated item in the sentence has a space after it. This was introduced by the line rewrap in this PR.
Suggested fix:
`Makefile`, `package.json` scripts, and similar build files.
| - Git hooks (`.git/hooks/`) | ||
| - CI configuration (`.github/workflows/`, `.gitlab-ci.yml`) | ||
| - IDE configuration (`.vscode/tasks.json`, `.idea/` run configurations) | ||
| - AI project configuration and settings (`.claude/`, `.codex/`, `.gemini/`) |
There was a problem hiding this comment.
[MEDIUM] AI tool config list may omit Cursor (.cursor/)
The new bullet covers Claude (.claude/), Codex (.codex/), and Gemini (.gemini/), but omits Cursor (.cursor/), which is among the most widely used AI coding assistants and has its own project-level configuration directory. Since this section warns users about files that could execute code automatically, leaving out a popular tool's config path could give users a false sense of safety.
Consider adding .cursor/ to both the summary bullet and the expanded description below it.
cf8d75b to
4bde608
Compare
2 participants
Description
The warnings are quite clear, and project-level AI tool configuration could conceivably be part of the IDE settings, but we could list them explicitly here
Reviews