chore: bump embedded PostgreSQL 18 to 18.3.0 - #568
Conversation
Upgrade fergusstrange/embedded-postgres from v1.33.0 to v1.34.0. The only library change is the V18 binary pin moving from 18.0.0 to 18.3.0; V14-V17 pins are unchanged. Update the dump fixture headers (Dumped from database version) and CLAUDE.md references to match. Verified on 18.3.0: full dump suite, diff suite, plan suite, and online/ integration tests. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Greptile SummaryUpdates
Confidence Score: 5/5The PR appears safe to merge, with no concrete changed-code defect identified in the dependency bump, documentation, or fixture updates. The dependency manifest and checksums consistently upgrade embedded PostgreSQL, while the remaining changes accurately synchronize documentation and cosmetic PostgreSQL dump headers. Important Files Changed
Reviews (1): Last reviewed commit: "chore: bump embedded PostgreSQL 18 to 18..." | Re-trigger Greptile |
There was a problem hiding this comment.
Pull request overview
This PR updates the project’s embedded PostgreSQL dependency to pick up the newer PostgreSQL 18.3.0 embedded binaries, and aligns documentation + dump fixtures to match the new reported server version string.
Changes:
- Bump
github.com/fergusstrange/embedded-postgresfromv1.33.0tov1.34.0(Go module + sums). - Update PG18 dump fixture headers from
PostgreSQL 18.0toPostgreSQL 18.3. - Refresh
CLAUDE.mdreferences to the embedded-postgres version.
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| testdata/dump/tenant/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/sakila/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_472_partition_clone_trigger/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_420_varchar_array_length_modifier/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_412_unique_nulls_not_distinct/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_396_check_constraint_is_not_null/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_345_array_cast/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_320_plpgsql_reserved_keyword_type/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_307_view_dependency_order/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_275_truncated_function_grants/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_252_function_schema_qualifier/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_191_function_procedure_overload/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/issue_125_function_default/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| testdata/dump/employee/pgschema.sql | Update fixture header to reflect PostgreSQL 18.3. |
| go.mod | Bump embedded-postgres requirement to v1.34.0. |
| go.sum | Update module sums for embedded-postgres v1.34.0. |
| CLAUDE.md | Update documentation references to embedded-postgres v1.34.0. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Upgrades
fergusstrange/embedded-postgresv1.33.0 → v1.34.0. I diffed the two library releases: the only code change is the V18 binary pin moving from18.0.0to18.3.0— the V14–V17 pins are untouched, so older-version test runs are unaffected.Also updated:
-- Dumped from database version PostgreSQL 18.0→18.3in the dump fixture headers (cosmetic — the dump tests normalize these lines away, but the fixtures stay honest)CLAUDE.mdTesting (all against the new 18.3.0 binaries)
TestDumpCommand, all 18 suites)internal/diffsuiteinternal/plansuiteonline/integration tests (TestPlanAndApply), which exercise the PG18 nativeNOT NULL ... NOT VALIDpath from feat: use native NOT NULL NOT VALID constraints on PostgreSQL 18+ #566 on 18.3🤖 Generated with Claude Code