google/comprehensive-rust

Explicitly call out dev-dependencies used in solutions

Geschlossen

#1.290 geöffnet am 02.10.2023

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (2.009 Forks)batch import
enhancementgood first issuehelp wanted

Repository-Metriken

Stars
 (32.954 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 9T 23h) (36 gemergte PRs in 30 T)

Beschreibung

From https://github.com/google/comprehensive-rust/pull/1288#issuecomment-1743131185: the safe FFI wrapper solution does not actually work when you copy-paste it into a new Cargo project. The problem is that the unit tests rely on tempfile.

We need to fix this: I suggest adding some text to the solution which shows the cargo add commands to run. Alternatively, we could {{#include ...}} the Cargo.toml file — but I fear this will become messy since all solutions currently share a single Cargo.toml file. We could perhaps split them up further?

Contributor Guide

Research-Richtung
Untersuche die Lösungsdatei unter exercises/day 3/solutions afternoon.html und deren Cargo.toml. Der Issue verweist auf PR #1288, der möglicherweise Diskussionen enthält. Erwäge, im Lösungstext einen Hinweis hinzuzufügen, der die erforderlichen `cargo add` Befehle für Dev Dependencies wie tempfile auflistet. Alternativ prüfe, ob das gemeinsame Cargo.toml aufgeteilt werden sollte, um Dev Dependencies pro Lösung einzubeziehen. Stelle sicher, dass die Lösung funktioniert, wenn sie in ein neues Projekt kopiert und eingefügt wird.
Tech Stack
rust
Domain
documentationdeveloper experiencetesting
Issue Type
Dokumentation
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust basicsCargo dev dependencies
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90