# code-review

> Team code review checklist - use for reviewing Python code quality, bugs, security issues, and best practices

- Author: Dan Wahlin
- Repository: reneenoble/github-copilot-cli-for-beginners
- Version: 20260209010909
- Stars: 0
- Forks: 0
- Last Updated: 2026-02-09
- Source: https://github.com/reneenoble/github-copilot-cli-for-beginners
- Web: https://mule.run/skillshub/@@reneenoble/github-copilot-cli-for-beginners~code-review:20260209010909

---

---
name: code-review
description: Team code review checklist - use for reviewing Python code quality, bugs, security issues, and best practices
---

# Code Review Skill

Apply this checklist when reviewing Python code.

## Code Quality Checklist

- [ ] All functions have type hints
- [ ] No bare except clauses
- [ ] No mutable default arguments
- [ ] Context managers used for file I/O
- [ ] Functions are under 50 lines
- [ ] Variable and function names follow PEP 8 (snake_case)

## Input Validation Checklist

- [ ] User input is validated before processing
- [ ] Edge cases handled (empty strings, None, out-of-range values)
- [ ] Error messages are clear and helpful

## Testing Checklist

- [ ] New code has corresponding pytest tests
- [ ] Edge cases are covered
- [ ] Tests use descriptive names

## Output Format

Present findings as:

```
## Code Review: [filename]

### Code Quality
- [PASS/FAIL] Description of finding

### Input Validation
- [PASS/FAIL] Description of finding

### Testing
- [PASS/FAIL] Description of finding

### Summary
[X] items need attention before merge
```