rjmurillo / EffectiveCSharp.Analyzers

Many of the recommendations in the book Effective C#: 50 Specific Ways to Improve Your C# can be validated by Roslyn-based analyzers and code fixes.
MIT License
2 stars 1 forks source link

Fix false positive for delegate #55

Closed rjmurillo closed 2 months ago

rjmurillo commented 2 months ago

When a delegate is used but not combined or passed around, ECS0700 was triggered. There is no problematic behavior. See #54 for details.

Resolves #54

codacy-production[bot] commented 2 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +0.11% (target: -1.00%) :white_check_mark: 94.12%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (28d90a7502be2fdbfe46670cd84de9cb8db5f2a0) | 811 | 735 | 90.63% | | | Head commit (1c24491df4191a524657abb2eea6afbcf92c39d7) | 821 (+10) | 745 (+10) | 90.74% (**+0.11%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#55) | 17 | 16 | **94.12%** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more