Skip to content
Build

Merge pull request #49 from EventStore/timothycoleman/add-qodana #211

Sign in to view logs

Merge pull request #49 from EventStore/timothycoleman/add-qodana

Merge pull request #49 from EventStore/timothycoleman/add-qodana #211

Triggered via push August 14, 2024 12:00
@timothycolemantimothycoleman
pushed 48dce9a
master
Status Success
Total duration 29s
Artifacts –

ci.yml

on: push
ci/github/scan-vulnerabilities
19s
ci/github/scan-vulnerabilities
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

28 warnings
ci/github/build-ubuntu-latest/net8.0/EventStore.Plugins.Tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
ci/github/scan-vulnerabilities
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Avoid using 'async' lambda when delegate type returns 'void': src/EventStore.Plugins/Plugin.cs#L157
Avoid using 'async' lambda when delegate type returns 'void'
Expression is always 'true' or 'false' according to nullable reference types' annotations: src/EventStore.Plugins/ConfigParser.cs#L34
Expression is always false according to nullable reference types' annotations
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L28
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L33
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L72
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L45
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L21
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L56
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L79
Constant 'string Resource' hides constant from outer class
Member hides static member from outer class: src/EventStore.Plugins/Authorization/Operations.cs#L41
Field 'OperationDefinition Options' hides field from outer class
Redundant argument with default value: src/EventStore.Plugins/Plugin.cs#L121
The parameter 'mode' has the same default value
Redundant argument with default value: src/EventStore.Plugins/Plugin.cs#L193
The parameter 'mode' has the same default value
Redundant using directive: src/EventStore.Plugins/Transforms/IChunkTransformFactory.cs#L1
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/ChunkDataReadStream.cs#L2
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/ChunkDataWriteStream.cs#L1
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/ChunkDataReadStream.cs#L1
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/IChunkWriteTransform.cs#L1
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/ChunkDataWriteStream.cs#L2
Using directive is not required by the code and can be safely removed
Redundant using directive: src/EventStore.Plugins/Transforms/IChunkTransformFactory.cs#L2
Using directive is not required by the code and can be safely removed
Auto-property accessor is never used (non-private accessibility): test/EventStore.Plugins.Tests/ConfigurationReaderTests/LdapsSettings.cs#L6
Auto-property accessor 'ValidateServerCertificate.set' is never used
Auto-property accessor is never used (non-private accessibility): test/EventStore.Plugins.Tests/ConfigurationReaderTests/LdapsSettings.cs#L18
Auto-property accessor 'RequireGroupMembership.set' is never used
Auto-property accessor is never used (non-private accessibility): test/EventStore.Plugins.Tests/ConfigurationReaderTests/LdapsSettings.cs#L9
Auto-property accessor 'AnonymousBind.set' is never used
Auto-property accessor is never used (private accessibility): src/EventStore.Plugins/Licensing/License.cs#L10
Auto-property accessor 'CurrentCultureIgnoreCase.set' is never used
Unused parameter (private accessibility): test/EventStore.Plugins.Tests/PluginBaseTests.cs#L215
Parameter 'x' is never used
Unused local variable: test/EventStore.Plugins.Tests/Diagnostics/PluginDiagnosticsDataCollectorTests.cs#L84
Local variable 'value' is never used
Unused local variable: test/EventStore.Plugins.Tests/Diagnostics/PluginDiagnosticsDataCollectorTests.cs#L49
Local variable 'value' is never used