Skip to content

feat(HNT-2055): add Claude PR review action, CLAUDE.md, and review-loop skill#1272

Merged
mmiermans merged 1 commit into
mainfrom
hnt-2055-claude-review
Mar 18, 2026
Merged

feat(HNT-2055): add Claude PR review action, CLAUDE.md, and review-loop skill#1272
mmiermans merged 1 commit into
mainfrom
hnt-2055-claude-review

Conversation

@mmiermans
Copy link
Copy Markdown
Contributor

@mmiermans mmiermans commented Mar 18, 2026

Goal

Set up Claude Code tooling for this repo: automated PR reviews, project context, and a shared review skill.

Reference

…op skill

Set up Claude Code tooling for this repo:
- GitHub Action for automated PR reviews (from content-monorepo)
- CLAUDE.md with project context, local dev setup, and test commands
- /review-loop skill for iterative code review (repo-level)
- Gitignore .claude/settings.local.json

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mmiermans mmiermans requested a review from a team as a code owner March 18, 2026 21:15
Comment thread CLAUDE.md
- **Port 3000** conflicts with `content-monorepo`'s Grafana/OTEL container if it's running; use `PORT=3001 npm start`.
- Use **npm**, not pnpm (ignore the untracked `pnpm-lock.yaml`).
- Login requires **Mozilla SSO** credentials.
- `npm start` opens a browser window by default — set `BROWSER=none` to suppress this (always do this unless the user explicitly asked to open the app).
Copy link
Copy Markdown
Contributor Author

@mmiermans mmiermans Mar 18, 2026

Choose a reason for hiding this comment

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

I tried to create a minimal CLAUDE.md file to provide additional context to Claude for this repo, and solve some "gotchas" that it ran into.

@mmiermans mmiermans merged commit 3e957cd into main Mar 18, 2026
7 checks passed
@mmiermans mmiermans deleted the hnt-2055-claude-review branch March 18, 2026 22:36
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