integrated-application-development / integrated-application-development/delphilint

Add option to analyze all VCS changed files

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
client companion enhancement server
Dominant language
Pascal
Stars
146
Forks
31
PR merge metrics
No merged PRs in 30d

Description

### Prerequisites

- [X] This improvement has not already been suggested.
- [X] This improvement would be generally useful, not specific to my code or setup.

### Engine area

Delphi IDE integration

### Improvement description

A common use case for DelphiLint is to scan changed files before committing them to source control, to ensure that any Sonar issues aren't inadvertently committed. To facilitate this, DelphiLint should add an option to "Analyze VCS Changed Files" in addition to the current "Analyze Active File" and "Analyze All Open Files".

This is a feature SonarLint already has, see IntelliJ:

![image](https://github.com/integrated-application-development/delphilint/assets/28214458/c127dcdc-7051-4510-a008-8ae5973a1e40)

### Rationale

It can be annoying to open and scan all changed files manually, and it has been proven convenient by its inclusion in SonarLint already.

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.