Removed Microsoft.Extensions.DependencyInjection from generated usings.#35
Removed Microsoft.Extensions.DependencyInjection from generated usings.#35
Conversation
Updated MethodGeneratorHelper and regenerated test files to exclude the directive.
There was a problem hiding this comment.
Pull request overview
This PR removes an unused using Microsoft.Extensions.DependencyInjection; directive from the code generator's output. The generator creates OpenApiHelper classes that do not use any types from this namespace, making the using directive unnecessary.
Changes:
- Removed the Microsoft.Extensions.DependencyInjection using directive from MethodGeneratorHelper.cs
- Regenerated 27 test snapshot files to reflect the updated generator output
- Bumped version from 7.0.0 to 7.0.1 in Directory.Build.props
Reviewed changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/AltaSoft.DomainPrimitives.Generator/Helpers/MethodGeneratorHelper.cs | Removed unused Microsoft.Extensions.DependencyInjection from the usings collection in AddOpenApiSchemas method |
| Directory.Build.props | Updated version to 7.0.1 and removed empty line between PropertyGroup elements |
| tests/.../Snapshots/*#OpenApiHelper.g.verified.cs (27 files) | Regenerated snapshot files reflecting the removal of the unused using directive |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <Version>7.0.0</Version> | ||
| <Version>7.0.1</Version> | ||
| </PropertyGroup> | ||
|
|
There was a problem hiding this comment.
An empty line was removed between the two PropertyGroup elements. This whitespace change appears unrelated to the stated purpose of this PR (removing Microsoft.Extensions.DependencyInjection from generated usings). Consider whether this whitespace change was intentional, and if not, restore the empty line for consistency with the existing file formatting.
Updated MethodGeneratorHelper and regenerated test files to exclude the directive.