Skip to content
Main build

Remove period from analyzer messages and inline in the analyzer classes #116

Sign in to view logs

Remove period from analyzer messages and inline in the analyzer classes

Remove period from analyzer messages and inline in the analyzer classes #116

Triggered via pull request June 12, 2024 23:55
@MattKotsenasMattKotsenas
opened #86
MattKotsenas:bugfix/analyzer-text
Status Success
Total duration 9m 32s
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#L67
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/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#L37
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#L35
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#L33
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 (ubuntu-22.04): Source/Moq.Analyzers.Test/ConstructorArgumentsShouldMatchAnalyzerTests.cs#L38
TODO "I think this _should_ fail, but currently passes. Tracked by #55." (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)
build (windows-2022): Source/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L67
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/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#L37
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#L35
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#L33
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
build (windows-2022): Source/Moq.Analyzers.Test/ConstructorArgumentsShouldMatchAnalyzerTests.cs#L38
TODO "I think this _should_ fail, but currently passes. Tracked by #55." (https://github.com/meziantou/Meziantou.Analyzer/blob/main/docs/Rules/MA0026.md)

Artifacts

Produced during runtime
Name Size
.NET Code Coverage Reports (ubuntu-22.04) Expired
697 KB
.NET Code Coverage Reports (windows-2022) Expired
697 KB
.NET Test Reports (ubuntu-22.04) Expired
66.4 KB
.NET Test Reports (windows-2022) Expired
66.2 KB
binlogs-ubuntu-22.04 Expired
566 KB
binlogs-windows-2022 Expired
587 KB
packages-ubuntu-22.04 Expired
51.7 KB
packages-windows-2022 Expired
52.1 KB