Skip to content

refactor: consolidate drizzle config files#202

Merged
mcowger merged 3 commits into
mainfrom
claude/issue-201-20260417-2032
Apr 17, 2026
Merged

refactor: consolidate drizzle config files#202
mcowger merged 3 commits into
mainfrom
claude/issue-201-20260417-2032

Conversation

@mcowger
Copy link
Copy Markdown
Owner

@mcowger mcowger commented Apr 17, 2026

This PR consolidates the duplicated PostgreSQL drizzle config files.

Changes:

  • Renamed drizzle.config.tsdrizzle.config.sqlite.ts for clarity
  • Removed duplicate drizzle.config.pg.ts (kept drizzle.config.postgres.ts as the canonical PostgreSQL config)
  • Updated test/setup.ts and CONTRIBUTING.md with new references

Fixes #201

Generated with Claude Code

github-actions Bot and others added 2 commits April 17, 2026 20:34
- Rename drizzle.config.ts to drizzle.config.sqlite.ts for clarity
- Remove duplicate drizzle.config.pg.ts (keeping the more complete
  drizzle.config.postgres.ts)
- Update all references in:
  - test/setup.ts
  - CONTRIBUTING.md

Note: Workflow files still reference drizzle.config.pg.ts and will need
to be manually updated after this PR is merged:
  - .github/workflows/generate-migrations.yml
  - .github/workflows/release.yml

Fixes #201

Co-authored-by: Matt Cowger <mcowger@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

Review Summary

Issue Found

CONTRIBUTING.md line 48-50: The paragraph "Then commit again with only your schema .ts files." is now orphaned. It originally followed the pre-commit hook instructions (lines 32-39), but the new "Drizzle Config Files" section was inserted between them, breaking the documentation flow.

Suggestion: Move this sentence up to follow line 39 (after the git reset HEAD command example) so it properly concludes the pre-commit hook section.

Otherwise

The changes look correct:

  • Config files renamed consistently (drizzle.config.pg.tsdrizzle.config.postgres.ts, drizzle.config.tsdrizzle.config.sqlite.ts)
  • All workflow references updated
  • dbCredentials commented out in postgres config as intended

@mcowger
Copy link
Copy Markdown
Owner Author

mcowger commented Apr 17, 2026

@claude implement the suggestion:

Issue Found
CONTRIBUTING.md line 48-50: The paragraph "Then commit again with only your schema .ts files." is now orphaned. It originally followed the pre-commit hook instructions (lines 32-39), but the new "Drizzle Config Files" section was inserted between them, breaking the documentation flow.

Suggestion: Move this sentence up to follow line 39 (after the git reset HEAD command example) so it properly concludes the pre-commit hook section.

@claude
Copy link
Copy Markdown

claude Bot commented Apr 17, 2026

Claude finished @mcowger's task in 35s —— View job


Done! I've moved the orphaned sentence to its proper place.

Changes Made

  • Moved "Then commit again with only your schema .ts files." from line 50 to line 41
  • Now properly follows the git reset HEAD command example in the pre-commit hook section
  • The documentation flow is restored: pre-commit hook section → Drizzle Config Files section → Code Style section

Tasks

  • Read CONTRIBUTING.md to understand the context
  • Move the orphaned sentence to follow the pre-commit hook section
  • Commit and push the changes

⚠️ Issues detected:

  • No branch was created — Claude may not have made any code changes

View job run

@mcowger mcowger merged commit 385818b into main Apr 17, 2026
@mcowger mcowger deleted the claude/issue-201-20260417-2032 branch April 17, 2026 20:46
github-actions Bot added a commit that referenced this pull request Apr 18, 2026
This PR consolidates the duplicated PostgreSQL drizzle config files.

**Changes:**
- Renamed `drizzle.config.ts` → `drizzle.config.sqlite.ts` for clarity
- Removed duplicate `drizzle.config.pg.ts` (kept
`drizzle.config.postgres.ts` as the canonical PostgreSQL config)
- Updated test/setup.ts and CONTRIBUTING.md with new references

Fixes #201

Generated with [Claude Code](https://claude.ai/code)

---------

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
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.

Duplicated Postgres Configs?

1 participant