Closed christianhelle closed 4 months ago
This pull request includes several code refactorings and improvements:
Removed an unused argument from the Should_Fail_Validating_V31_Spec method.
Should_Fail_Validating_V31_Spec
Added tests for invalid OpenAPI specs.
Moved the check for NoLogging into the Analytics.Configure method.
NoLogging
Analytics.Configure
Extracted the WriteFiles() method.
WriteFiles()
Removed an unused CheckForDuplicateOperationIds() method.
CheckForDuplicateOperationIds()
Enabled TreatWarningsAsErrors globally in the project file.
TreatWarningsAsErrors
Removed duplicate [InlineData] attributes.
[InlineData]
Removed an unused OutputType argument.
OutputType
These changes aim to improve code quality, remove unused code, and enable stricter error handling by treating warnings as errors.
Issues 0 New issues 0 Accepted issues
Measures 0 Security Hotspots 0.0% Coverage on New Code 0.0% Duplication on New Code
See analysis details on SonarCloud
This pull request includes several code refactorings and improvements:
Removed an unused argument from the
Should_Fail_Validating_V31_Spec
method.Added tests for invalid OpenAPI specs.
Moved the check for
NoLogging
into theAnalytics.Configure
method.Extracted the
WriteFiles()
method.Removed an unused
CheckForDuplicateOperationIds()
method.Enabled
TreatWarningsAsErrors
globally in the project file.Removed duplicate
[InlineData]
attributes.Removed an unused
OutputType
argument.These changes aim to improve code quality, remove unused code, and enable stricter error handling by treating warnings as errors.