crytic/slither

Filter is_constructor_variables on printers

Open

#865 geöffnet am 14. Juni 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (886 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (4.769 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 54T 9h) (2 gemergte PRs in 30 T)

Beschreibung

Some printers still output info if is_constructor_variables is true.

We should review every printer to see if it makes sense to print them

Contributor Guide

Research-Richtung
Überprüfe jeden Drucker in der Druckersammlung von Slither (z. B. im Verzeichnis slither/printers/). Prüfe für jeden Drucker, ob er Variablen ausgibt, wenn `is constructor variables` true ist. Bestimme, ob das Ausgeben solcher Variablen für diesen Drucker angemessen ist. Falls nicht, füge eine Bedingung hinzu, um sie zu überspringen. Siehe die vorhandene Logik in slither/core/declarations/function.py für die Eigenschaft `is constructor variables`.
Tech Stack
python
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75