integrated-application-development / integrated-application-development/sonar-delphi

Split `Unused*` rules into implementation and interface versions

Open
#343 0 comments 0 reactions 0 assignees View on GitHub
enhancement rule
Dominant language
Java
Stars
159
Forks
31
Avg merge
5d 4h
Merged PRs (30d)
4

Description

### Prerequisites

- [x] This improvement has not already been suggested.
- [x] This improvement should not be implemented as a separate rule.

### Rule to improve

Unused*

### Improvement description

Split each `Unused*` rule (e.g. `UnusedConstant`, `UnusedField`, etc.) into an implementation and interface variant.

### Rationale

Currently [integrated-application-development/delphilint](https://github.com/integrated-application-development/delphilint/) chooses not to run the `Unused*` rules in its analysis, because it's so likely to be inaccurate when only scanning a subset of the source files for a project.

If the rules were divided into 'public' and 'private' variants, then delphilint would be able to run the 'implementation' variants and confidently report unused local variables, constants, fields, methods, etc.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.