improve collapsible if in complex cases with same expressions

Repository-Metriken

Stars

10.406 Sterne (10.406 Sterne)

PR-Merge-Metriken

Durchschn. Merge 19T 22h (Durchschn. Merge 19T 22h)113 gemergte PRs in 30 T (113 gemergte PRs in 30 T)

Beschreibung

if a {
    if b {
        f();
    }
} else {
    f();
}

if !a || b {
    f();
}

Contributor Guide

Research-Richtung

Untersuche die aktuelle Implementierung des collapsible if Lints in rust clippy (Datei clippy lints/src/collapsible if.rs). Verstehe, wie es derzeit einfache verschachtelte if Anweisungen erkennt. Die Erweiterung erfordert das Hinzufügen von Logik, um Fälle zu erkennen, in denen der else Zweig denselben Ausdruck wie der if Körper enthält, und diese in eine einzelne Bedingung umzuwandeln. Suche nach bestehenden Diskussionen oder Versuchen (das Issue ist zugewiesen, aber es gibt keinen PR). Der Maintainer könnte Input zum gewünschten Ansatz haben. Konzentriere dich auf die AST Muster für das gegebene Beispiel.

Tech Stack

rust

Domain

tooling

Issue Type

Funktion

SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.

3

Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.

1-2 Tage

AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.

Aktiv

KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.

Klar

Voraussetzungen

Rust

EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.

65