Skip to content
Main build

Fix SA0001: XML comment analysis is disabled due to project configuration #118

Sign in to view logs

Fix SA0001: XML comment analysis is disabled due to project configuration

Fix SA0001: XML comment analysis is disabled due to project configuration #118

Triggered via pull request June 13, 2024 01:31
@MattKotsenasMattKotsenas
opened #87
MattKotsenas:refactor/warnings-as-errors
Status Success
Total duration 6m 26s
Artifacts 8

main.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
build (ubuntu-22.04): Source/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L72
Possible null reference argument for parameter 'expression' in 'TypeInfo CSharpExtensions.GetTypeInfo(SemanticModel? semanticModel, ExpressionSyntax expression, CancellationToken cancellationToken = default(CancellationToken))'.
build (ubuntu-22.04): Source/Moq.Analyzers/Helpers.cs#L80
Possible null reference argument for parameter 'item' in 'void List<T>.Add(T item)'.
build (ubuntu-22.04): Source/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L30
Missing XML comment for publicly visible type or member 'ConstructorArgumentsShouldMatchAnalyzer.Initialize(AnalysisContext)'
build (ubuntu-22.04): Source/Moq.Analyzers/Helpers.cs#L93
TODO Check what is the best way to do such checks (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (ubuntu-22.04): Source/Moq.Analyzers/NoConstructorArgumentsForInterfaceMockAnalyzer.cs#L42
TODO Think how to make this piece more elegant while fast (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (ubuntu-22.04): Source/Moq.Analyzers/NoSealedClassMocksAnalyzer.cs#L40
TODO Think how to make this piece more elegant while fast (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (ubuntu-22.04): Source/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L37
Method is too long (72 lines; maximum allowed: 60) (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0051.md)
build (ubuntu-22.04): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L12
TODO .As<BaseSampleClass> and .As<SampleClass> feels redundant (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (ubuntu-22.04): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L16
TODO Testing with .Setup() and .Returns() seems unnecessary. (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (ubuntu-22.04): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L16
build (windows-2022): Source/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L72
Possible null reference argument for parameter 'expression' in 'TypeInfo CSharpExtensions.GetTypeInfo(SemanticModel? semanticModel, ExpressionSyntax expression, CancellationToken cancellationToken = default(CancellationToken))'.
build (windows-2022): Source/Moq.Analyzers/Helpers.cs#L80
Possible null reference argument for parameter 'item' in 'void List<T>.Add(T item)'.
build (windows-2022): Source/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L30
Missing XML comment for publicly visible type or member 'ConstructorArgumentsShouldMatchAnalyzer.Initialize(AnalysisContext)'
build (windows-2022): Source/Moq.Analyzers/Helpers.cs#L93
TODO Check what is the best way to do such checks (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers/NoConstructorArgumentsForInterfaceMockAnalyzer.cs#L42
TODO Think how to make this piece more elegant while fast (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers/NoSealedClassMocksAnalyzer.cs#L40
TODO Think how to make this piece more elegant while fast (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers/ConstructorArgumentsShouldMatchAnalyzer.cs#L37
Method is too long (72 lines; maximum allowed: 60) (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0051.md)
build (windows-2022): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L12
TODO .As<BaseSampleClass> and .As<SampleClass> feels redundant (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L16
TODO Testing with .Setup() and .Returns() seems unnecessary. (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers.Test/AsAcceptOnlyInterfaceAnalyzerTests.cs#L16

Artifacts

Produced during runtime
Name Size
.NET Code Coverage Reports (ubuntu-22.04) Expired
698 KB
.NET Code Coverage Reports (windows-2022) Expired
698 KB
.NET Test Reports (ubuntu-22.04) Expired
66.4 KB
.NET Test Reports (windows-2022) Expired
66.4 KB
binlogs-ubuntu-22.04 Expired
567 KB
binlogs-windows-2022 Expired
588 KB
packages-ubuntu-22.04 Expired
51.8 KB
packages-windows-2022 Expired
52.1 KB