Security concern

**Avoid:**
- "This is wrong" → "Consider this alternative approach..."
- "Bad code" → "This could be improved by..."
- "Fix this" → "What do you think about..."

#### 3. Review Types

```bash
# Approve PR
"LGTM! Great work on the responsive design. The hover effects are a nice touch."

# Request changes
"The functionality looks good, but I have a few suggestions for code organization. Please see my inline comments."

# Comment only
"Nice implementation! I left a few optional suggestions for future improvements."

# Address feedback with new commits
git checkout feature/add-projects-section

# Make requested changes
echo ":root { --spacing-lg: 2rem; }" >> style.css
sed -i 's/gap: 2rem/gap: var(--spacing-lg)/' style.css

# Commit improvements
git add style.css
git commit -m "Use CSS custom properties for consistent spacing

Addresses review feedback about maintainable spacing values."

# Push updates
git push origin feature/add-projects-section

> Why did you choose CSS Grid over Flexbox here?

Good question! I chose CSS Grid because:

1. It handles both rows and columns automatically
2. The `auto-fit` and `minmax()` functions provide better responsive behavior
3. It's more semantic for this card-based layout

Flexbox would work too, but would require more media queries for the responsive behavior.

# Create draft PR for early feedback
# On GitHub: Check "Create draft pull request"
# Or use GitHub CLI:
gh pr create --draft --title "WIP: Add projects section" --body "Early version for feedback"

## Description

Brief description of changes

## Type of Change

- [ ] Bug fix
- [ ] New feature
- [ ] Breaking change
- [ ] Documentation update

## Testing

- [ ] Unit tests pass
- [ ] Integration tests pass
- [ ] Manual testing completed

## Checklist

- [ ] Code follows style guidelines
- [ ] Self-review completed
- [ ] Documentation updated
- [ ] No breaking changes (or documented)

## Screenshots (if applicable)

## Additional Notes

## Description

Implements user authentication system

Closes #123
Fixes #456
Resolves #789

# When pair programming
git commit -m "Add user authentication

Co-authored-by: Jane Doe <jane@example.com>"

# Windows
winget install GitHub.cli

# macOS
brew install gh

# Linux
sudo apt install gh

# Login to GitHub
gh auth login

# Check status
gh auth status

# Create PR
gh pr create --title "Add projects section" --body "Adds responsive projects showcase"

# Create draft PR
gh pr create --draft

# List PRs
gh pr list
gh pr list --state open
gh pr list --author @me

# View PR
gh pr view 123
gh pr view --web  # Open in browser

# Check out PR locally
gh pr checkout 123

# Review PR
gh pr review 123 --approve
gh pr review 123 --request-changes --body "Please fix the CSS issues"

# Merge PR
gh pr merge 123
gh pr merge 123 --squash
gh pr merge 123 --rebase

# Close PR
gh pr close 123

# Good: Small, focused PRs
git log --oneline main..feature/add-button  # 2-3 commits

# Avoid: Large, multi-purpose PRs
git log --oneline main..feature/redesign-everything  # 20+ commits

# Good PR Description

## What

Adds user authentication with JWT tokens

## Why

Users need to log in to access personalized features

## How

- Implemented login/logout endpoints
- Added JWT middleware for protected routes
- Created user registration flow

## Testing

- Added unit tests for auth functions
- Tested login flow manually
- Verified token expiration handling

# Before creating PR, clean up commits
git rebase -i HEAD~5

# Squash related commits
# Fix commit messages
# Remove debug commits

# Review your own changes before creating PR
git diff main..feature/branch

# Check for:
# - Debug code left behind
# - TODO comments
# - Formatting issues
# - Missing documentation

# Creates merge commit preserving branch history
git checkout main
git merge feature/branch

*   a1b2c3d Merge pull request #123 from feature/branch
|\
| * d4e5f6g Add projects section
| * g7h8i9j Update navigation
|/
* j0k1l2m Previous commit

# Combines all commits into single commit
# Available on GitHub PR interface

* a1b2c3d Add projects section (#123)
* j0k1l2m Previous commit

# Replays commits without merge commit
git checkout main
git rebase feature/branch

* d4e5f6g Add projects section
* g7h8i9j Update navigation
* j0k1l2m Previous commit

# .github/workflows/pr-checks.yml
name: PR Checks
on:
  pull_request:
    branches: [main]

jobs:
  test:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - name: Setup Node.js
        uses: actions/setup-node@v3
        with:
          node-version: "18"
      - name: Install dependencies
        run: npm install
      - name: Run tests
        run: npm test
      - name: Run linter
        run: npm run lint
      - name: Check formatting
        run: npm run format:check

# Configure on GitHub:
# Settings → Branches → Add rule

# Require:
# - Status checks to pass
# - Up-to-date branches
# - Review from code owners
# - Signed commits

# Update feature branch with latest main
git checkout feature/branch
git fetch origin
git rebase origin/main

# Resolve conflicts if any
# Push updated branch
git push --force-with-lease origin feature/branch

# Fix issues locally
npm run lint:fix
npm test

# Commit fixes
git add .
git commit -m "Fix linting issues"
git push origin feature/branch

# Check reflog for lost commits
git reflog

# Recover if needed
git reset --hard commit-hash
git push --force-with-lease origin feature/branch

# PR workflow
git checkout -b feature/branch    # Create feature branch
# ... make changes ...
git push -u origin feature/branch # Push to GitHub
# Create PR on GitHub interface

# GitHub CLI
gh pr create                      # Create PR
gh pr list                        # List PRs
gh pr checkout 123               # Checkout PR locally
gh pr review 123 --approve       # Approve PR
gh pr merge 123 --squash         # Merge PR

# Update PR
git add .
git commit -m "Address feedback"
git push origin feature/branch

# Clean up after merge
git checkout main
git pull origin main
git branch -d feature/branch