Move EventStore.Plugins.Connectors to main repo. #4960
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
removed whitespace added roslynator analysers refactored activation a bit
not building yet
builds again but still broken
…ams when a connector is created or deleted
…ation command tests
Deploying eventstore with
|
| Latest commit: |
cf0379a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5576ec02.eventstore.pages.dev |
| Branch Preview URL: | https://yoeight-integ-connectors.eventstore.pages.dev |
Qodana for .NET36 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
Deleted unnecessary .editorconfig and .gitignore files. Removed any config related to connectors from directory.build.props Enhanced connectors project setup with updated nullable settings, deterministic builds, suppressed warnings, and improved resource handling. Simplified unit test node generation logic using Guid instead of NewId, removing another package dependency.
…jects. The entire connector assignment implementation, including strategies like LeastLoadedWithAffinity and RoundRobin, and their related interfaces, classes, and tests, was deleted. This reflects a shift away from the use of these assignment mechanisms in the system.
… namespaces Removed `KurrentDB.Toolkit.Testing.Http` project and moved its components under `KurrentDB.Connectors.Tests.Infrastructure.Http`. Updated references and namespaces to align with the new structure for improved organization and maintainability.
…testing code Eliminated unused models (ClusterNode, ClusterNodeId, ClusterTopology) and their references. Deprecated test configurations and fixtures were commented out or relocated. Updated namespaces for testing utilities to align with the new Surge library structure.
timothycoleman
approved these changes
May 6, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.