Skip to content

Conversation

@HofmeisterAn
Copy link
Collaborator

What does this PR do?

EventStoreDb got replaced by KurrentDB: https://www.kurrent.io/blog/kurrent-re-brand-faq.

Why is it important?

-

Related issues

@HofmeisterAn HofmeisterAn added breaking change Causing compatibility issues chore A change that doesn't impact the existing functionality, e.g. internal refactorings or cleanups labels Dec 7, 2025
@netlify
Copy link

netlify bot commented Dec 7, 2025

Deploy Preview for testcontainers-dotnet ready!

Name Link
🔨 Latest commit 40a652d
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-dotnet/deploys/6935b2374cbc810008453ae8
😎 Deploy Preview https://deploy-preview-1599--testcontainers-dotnet.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link

coderabbitai bot commented Dec 7, 2025

Summary by CodeRabbit

  • Chores
    • Removed EventStoreDb testcontainer module and all associated tests from the library. The EventStoreDb container support is no longer available.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

The PR removes the entire Testcontainers.EventStoreDb module from the solution, including all source files, test project files, project configuration files, and references in the solution file. No other projects or modules are affected.

Changes

Cohort / File(s) Summary
Solution & Project Configuration
Testcontainers.sln, src/Testcontainers.EventStoreDb/Testcontainers.EventStoreDb.csproj, tests/Testcontainers.EventStoreDb.Tests/Testcontainers.EventStoreDb.Tests.csproj
Removed EventStoreDb project entries, build configurations, and test project configuration from solution file and project files.
EventStoreDb Source Code
src/Testcontainers.EventStoreDb/EventStoreDbBuilder.cs, src/Testcontainers.EventStoreDb/EventStoreDbContainer.cs, src/Testcontainers.EventStoreDb/EventStoreDbConfiguration.cs, src/Testcontainers.EventStoreDb/Usings.cs, src/Testcontainers.EventStoreDb/.editorconfig
Deleted all EventStoreDb library source files including builder, container, configuration classes, global usings, and editor configuration.
EventStoreDb Test Project
tests/Testcontainers.EventStoreDb.Tests/EventStoreDbContainerTest.cs, tests/Testcontainers.EventStoreDb.Tests/Usings.cs, tests/Testcontainers.EventStoreDb.Tests/.editorconfig, tests/Testcontainers.EventStoreDb.Tests/.runs-on, tests/Testcontainers.EventStoreDb.Tests/Dockerfile
Removed all test source files, test usings, editor configuration, runner specification, and Dockerfile for the EventStoreDb test project.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Verify solution file is properly cleaned of all EventStoreDb project entries and nested project references
  • Confirm no other projects retain dependencies on the removed EventStoreDb module
  • Ensure complete removal across solution, source, and test directories

Possibly related PRs

  • PR #1583: Marks EventStoreDbBuilder as obsolete, providing context for this full module removal

Suggested labels

module

Poem

A module bids its final goodbye,
EventStoreDb fades into the sky,
Tests and builders vanish with grace,
Leaving clean code in its place. 🐰✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description covers the mandatory 'What does this PR do?' section with a clear explanation. However, the 'Why is it important?' section only contains a dash placeholder rather than substantive rationale, and 'How to test this PR' section is missing. Expand 'Why is it important?' with substantive context (e.g., KurrentDB migration timeline, user impact) and add testing guidance under 'How to test this PR'.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: Remove EventStoreDb module' accurately summarizes the main change—removing the entire EventStoreDb module from the codebase, which aligns with the raw summary showing deletion of all related files and project references.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/remove-event-store-db

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d2318d1 and 40a652d.

📒 Files selected for processing (13)
  • Testcontainers.sln (0 hunks)
  • src/Testcontainers.EventStoreDb/.editorconfig (0 hunks)
  • src/Testcontainers.EventStoreDb/EventStoreDbBuilder.cs (0 hunks)
  • src/Testcontainers.EventStoreDb/EventStoreDbConfiguration.cs (0 hunks)
  • src/Testcontainers.EventStoreDb/EventStoreDbContainer.cs (0 hunks)
  • src/Testcontainers.EventStoreDb/Testcontainers.EventStoreDb.csproj (0 hunks)
  • src/Testcontainers.EventStoreDb/Usings.cs (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/.editorconfig (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/.runs-on (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/Dockerfile (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/EventStoreDbContainerTest.cs (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/Testcontainers.EventStoreDb.Tests.csproj (0 hunks)
  • tests/Testcontainers.EventStoreDb.Tests/Usings.cs (0 hunks)
💤 Files with no reviewable changes (13)
  • src/Testcontainers.EventStoreDb/.editorconfig
  • tests/Testcontainers.EventStoreDb.Tests/.runs-on
  • tests/Testcontainers.EventStoreDb.Tests/.editorconfig
  • tests/Testcontainers.EventStoreDb.Tests/EventStoreDbContainerTest.cs
  • src/Testcontainers.EventStoreDb/Usings.cs
  • src/Testcontainers.EventStoreDb/EventStoreDbContainer.cs
  • tests/Testcontainers.EventStoreDb.Tests/Usings.cs
  • Testcontainers.sln
  • src/Testcontainers.EventStoreDb/Testcontainers.EventStoreDb.csproj
  • tests/Testcontainers.EventStoreDb.Tests/Dockerfile
  • src/Testcontainers.EventStoreDb/EventStoreDbBuilder.cs
  • tests/Testcontainers.EventStoreDb.Tests/Testcontainers.EventStoreDb.Tests.csproj
  • src/Testcontainers.EventStoreDb/EventStoreDbConfiguration.cs

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@HofmeisterAn HofmeisterAn merged commit ed192fb into develop Dec 7, 2025
13 checks passed
@HofmeisterAn HofmeisterAn deleted the feature/remove-event-store-db branch December 7, 2025 17:40
@digital88
Copy link
Contributor

FYI .slnx file still has this project reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Causing compatibility issues chore A change that doesn't impact the existing functionality, e.g. internal refactorings or cleanups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants