Skip to content

Conversation

@deepsource-autofix
Copy link
Contributor

This PR removes various unused variable declarations throughout the codebase to improve readability and eliminate compiler warnings. By deleting these unreferenced variables, the changes streamline the code and maintain cleaner standards.

  • Variable is declared but not used: In multiple test and infrastructure files, variables such as AggregateNameSetEvent aggregateNameSetEvent, Func<String, Int32, ISubscriptionRepository> subscriptionRepositoryResolver, TestAggregate instances, IConfigurationBuilder configurationBuilder, IPAddress[] ipAddresses, StringBuilder builder, Stopwatch stopwatch, DockerServices sharedDockerServices, and TimeSpan retryTimeout were declared without being used. We removed these unused declarations to clean up the code, eliminate compiler warnings, and improve maintainability.

This Autofix was generated by AI. Please review the change before merging.

This PR removes various unused variable declarations throughout the codebase to improve readability and eliminate compiler warnings. By deleting these unreferenced variables, the changes streamline the code and maintain cleaner standards.

- Variable is declared but not used: In multiple test and infrastructure files, variables such as AggregateNameSetEvent aggregateNameSetEvent, Func<String, Int32, ISubscriptionRepository> subscriptionRepositoryResolver, TestAggregate instances, IConfigurationBuilder configurationBuilder, IPAddress[] ipAddresses, StringBuilder builder, Stopwatch stopwatch, DockerServices sharedDockerServices, and TimeSpan retryTimeout were declared without being used. We removed these unused declarations to clean up the code, eliminate compiler warnings, and improve maintainability.

> This Autofix was generated by AI. Please review the change before merging.
@deepsource-io
Copy link

deepsource-io bot commented Sep 19, 2025

Here's the code health analysis summary for commits 686b774..6900608. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource SQL LogoSQL✅ SuccessView Check ↗
DeepSource Docker LogoDocker✅ SuccessView Check ↗
DeepSource C# LogoC#✅ Success
🎯 19 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@StuartFerguson StuartFerguson merged commit 4c13e8e into master Sep 19, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants