Skip to content

184#188

Merged
RAprogramm merged 3 commits into
mainfrom
184-impl
Jul 4, 2026
Merged

184#188
RAprogramm merged 3 commits into
mainfrom
184-impl

Conversation

@RAprogramm

Copy link
Copy Markdown
Owner

Follow-up to #184 (found by re-validating against a real consumer): the repository impl block and the constraints mapper were still wrapped in consumer-side #[cfg(feature = "postgres")], so a consumer without a declared postgres feature kept getting unexpected_cfgs warnings and silently lost the impl.

  • postgres::generate now returns nothing when the facade's postgres feature is off and emits the impl plainly when it is on
  • Constraints mapper cfg removed (already inside the gated generator)
  • Module docs updated

Suites green on default, postgres, and --all-features.

@codecov

codecov Bot commented Jul 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ates/entity-derive-impl/src/entity/sql/postgres.rs 66.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@RAprogramm RAprogramm merged commit f54629b into main Jul 4, 2026
14 of 15 checks passed
@RAprogramm RAprogramm deleted the 184-impl branch July 4, 2026 08:10
@RAprogramm RAprogramm mentioned this pull request Jul 4, 2026
Merged
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.

1 participant