Skip to content

Feature/rework scaffolding#55

Merged
sebastian-ederer merged 12 commits into
mainfrom
feature/rework_scaffolding
Jul 23, 2026
Merged

Feature/rework scaffolding#55
sebastian-ederer merged 12 commits into
mainfrom
feature/rework_scaffolding

Conversation

@sebastian-ederer

Copy link
Copy Markdown
Member

Description

The PR reworks the entire scaffolding process to generate code that uses actual Data Annotation Attributes or Fluent API calls instead of spamming .HasAnnotation(...) calls when scaffolding a DbContext.

Related Issues

Type of Change

  • Bug fix (non-breaking change adressing an issue)
  • New feature (non-breaking change adding functionality)
  • Refactoring
  • Documentation
  • CI / build / tooling
  • Breaking change (fix or feature causing existing functionality to change)

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated (Testcontainers)
  • Existing tests pass (dotnet test)

Checklist

  • Code follows the project's coding styles and guidelines
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • No new compiler warnings introduced
  • Public API changes have XML documentation

@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.45355% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...sign/Scaffolding/HypertableScaffoldingExtractor.cs 89.65% 3 Missing ⚠️
...c/Eftdb.Design/Scaffolding/ViewDefinitionParser.cs 97.65% 3 Missing ⚠️
src/Eftdb.Design/TimescaleDatabaseModelFactory.cs 86.95% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@sebastian-ederer
sebastian-ederer force-pushed the feature/rework_scaffolding branch from ee4e208 to 37a7c5f Compare July 23, 2026 16:35
@sebastian-ederer
sebastian-ederer merged commit f03c669 into main Jul 23, 2026
3 checks passed
@sebastian-ederer
sebastian-ederer deleted the feature/rework_scaffolding branch July 23, 2026 16:44
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