Conversation
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Pull request overview
Adds initial Claude Code configuration and repository guidance for working in the CDP monorepo.
Changes:
- Introduces
CLAUDE.mdwith an overview of CDP, repo structure, and preferred engineering patterns. - Adds a committed
.claude/settings.jsonenabling TypeScript and Python LSP plugins. - Updates
.gitignoreto trackCLAUDE.mdand ignore only local Claude settings overrides.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
CLAUDE.md |
Adds project/context documentation and preferred patterns for contributors/assistants. |
.gitignore |
Stops ignoring CLAUDE.md and narrows Claude ignores to local-only settings. |
.claude/settings.json |
Adds shared Claude Code plugin configuration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
joanagmaia
left a comment
There was a problem hiding this comment.
Given the size of the codebase it might make sense to be more specific on some additional practices on the backend. Also to add more coverage to frontend structure.
But I think this is already a good way to start. Once we merge let's all contribute it 👌
|
|
||
| ## Tech stack | ||
|
|
||
| TypeScript, Node.js, Express, PostgreSQL (pg-promise), Temporal, Kafka, Redis, OpenSearch, Zod, Bunyan, AWS S3. |
There was a problem hiding this comment.
Missing frontend tech stack. At least Vue should be added here.
There was a problem hiding this comment.
Oops, sorry, I missed it (the tech stack was auto-generated by the agent).
Let me add and merge the pr 👌🏻
Co-authored-by: Mouad BANI <mbani@contractor.linuxfoundation.org> Signed-off-by: Yeganathan S <63534555+skwowet@users.noreply.github.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
+1, we should definitely refine the |
Note
Low Risk
Docs and developer-tooling configuration only; no runtime or production code paths are changed.
Overview
Adds initial Claude Code setup by committing
.claude/settings.json(enabling the official TypeScript and Pyright LSP plugins) and a newCLAUDE.mdwith repository-specific development guidelines for AI-assisted work.Updates
.gitignoreto stop ignoringCLAUDE.md/the whole.claudefolder and instead ignore only local Claude artifacts (e.g.,.claude/settings.local.json, caches, logs, sessions, todos).Written by Cursor Bugbot for commit 9d2d8e4. This will update automatically on new commits. Configure here.