Byron/trash-rs

Use references in `restore_all` and `purge_all`

Offen

#54 geöffnet am 17.07.2022

Auf GitHub ansehen
 (12 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (40 Forks)auto 404
enhancementgood first issuehelp wanted

Repository-Metriken

Stars
 (247 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

restore_all and purge_all should take references because they don't need the ownership of TrashItems. This could reduce cloning when calling the functions. restore_all might need an additional clone on a RestoreCollision error because there is remaining_items. My use case is specified in #53 .

Contributor Guide

Research-Richtung
Ändere `restore all` und `purge all`, um `&[TrashItem]` anstelle von `Vec<TrashItem>` zu akzeptieren. Passe den internen Code an, um mit Referenzen zu arbeiten, und behandle den `RestoreCollision` Fehler, indem du bei Bedarf die verbleibenden Elemente klonst.
Tech Stack
rust
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.
1-3 Stunden
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.
75