Skip to content

Conversation

@deepsource-autofix
Copy link
Contributor

This PR refactors test classes to leverage C# 10 file-scoped namespaces. It relocates using directives to the top of each file and removes unnecessary region directives for improved readability and consistency across the codebase.

  • Use File Scoped namespaces instead of typical namespaces: Legacy block-scoped namespace declarations in Shared.EventStore.Tests and Shared.Tests were converted to file-scoped namespaces at the top of each file. All using directives were consolidated above the namespace, and extraneous #region blocks were removed to streamline the code structure and adhere to modern C# conventions.

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

This PR refactors test classes to leverage C# 10 file-scoped namespaces. It relocates using directives to the top of each file and removes unnecessary region directives for improved readability and consistency across the codebase.

- Use File Scoped `namespace`s instead of typical `namespace`s: Legacy block-scoped namespace declarations in Shared.EventStore.Tests and Shared.Tests were converted to file-scoped namespaces at the top of each file. All using directives were consolidated above the namespace, and extraneous #region blocks were removed to streamline the code structure and adhere to modern C# conventions.

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

deepsource-io bot commented Sep 20, 2025

Here's the code health analysis summary for commits fcda5f7..2cf7371. View details on DeepSource ↗.

Analysis Summary

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

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

@StuartFerguson StuartFerguson merged commit f3bc955 into master Sep 20, 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