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

Add rule ECS0007 #42

Closed rjmurillo closed 2 months ago

rjmurillo commented 2 months ago

Related: https://github.com/BillWagner/EffectiveCSharpAnalyzers/issues/7 Closes #32

The problem addressed by this rule is the use of methods that should involve callback mechanisms without employing delegates for this purpose. Specifically, methods are being invoked directly without the use of delegate types such as Predicate<T>, Action<T>, or Func<T>, which are designed to handle callbacks in a type-safe and consistent manner.

CLAassistant commented 2 months ago

CLA assistant check
All committers have signed the CLA.

codacy-production[bot] commented 2 months ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +1.14% (target: -1.00%) :white_check_mark: 93.33%
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (dc194bbde030e9c40d8d9cdb1e0b5ff8919fe5a8) | 439 | 356 | 81.09% | | | Head commit (6a7bd0009329704bf42ae9ea5173403f5f0bfbb1) | 484 (+45) | 398 (+42) | 82.23% (**+1.14%**) | **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 (#42) | 45 | 42 | **93.33%** | **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