Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moq1100 incorrectly firing on nullable parameters #187

Merged
merged 20 commits into from
Aug 28, 2024

Update SquiggleCop baseline

d028e35
Select commit
Loading
Failed to load commit list.
Merged

Moq1100 incorrectly firing on nullable parameters #187

Update SquiggleCop baseline
d028e35
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Aug 27, 2024 in 0s

2 new issues (0 max.) of at least medium severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 3
           

Complexity decreasing per file
==============================
+ src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs  -2

See the complete overview on Codacy

Annotations

Check failure on line 109 in src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L109 

The Cyclomatic Complexity of this method is 11 which is greater than 10 authorized.

Check failure on line 127 in src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L127 

Add a 'default' clause to this 'switch' statement.

Check notice on line 127 in src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

src/Moq.Analyzers/CallbackSignatureShouldMatchMockedMethodAnalyzer.cs#L127 

Replace this 'switch' statement with 'if' statements to increase readability.
View more details on Codacy Production