Add code review guidelines to SKILL.md - #8
Conversation
This document outlines guidelines for conducting thorough code reviews, focusing on security, code quality, testing, and architecture practices.
![amazon-q-developer[bot]](https://avatars.githubusercontent.com/in/1220912?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR adds code review guidelines documentation to the repository. The file is well-structured with clear sections covering security, code quality, testing, and review output format. No blocking issues found - the documentation is ready to merge.
You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.
There was a problem hiding this comment.
🟢 Approval recommended
The change is a straightforward documentation-only addition with no functional or security impact.
Pull request overview
Adds a new code-review Skill document under .github/skills/ that standardizes how reviewers should approach PR reviews in this repository context.
Changes:
- Introduces a
code-reviewskill definition (YAML front matter) with name/description metadata. - Documents review guidelines covering security/authentication, code quality/architecture, and testing expectations.
- Defines a suggested structure for review feedback output.
File summaries
| File | Description |
|---|---|
| .github/skills/code-review/SKILL.md | Adds the code review skill documentation and expected review focus areas/output structure. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This document outlines guidelines for conducting thorough code reviews, focusing on security, code quality, testing, and architecture practices.