google/comprehensive-rust

Explicitly call out dev-dependencies used in solutions

Chiusa

#1290 aperta il 2 ott 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (2009 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (32.954 stelle)
Metriche merge PR
 (Merge medio 9g 23h) (36 PR mergiate in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Esamina il file della soluzione in exercises/day 3/solutions afternoon.html e il suo Cargo.toml. L'issue fa riferimento alla PR #1288, che potrebbe contenere discussioni. Considera di aggiungere una nota nel testo della soluzione che elenchi i comandi `cargo add` necessari per le dev dependencies come tempfile. In alternativa, valuta la possibilità di suddividere il Cargo.toml condiviso per includere le dev dependencies specifiche per ogni soluzione. Assicurati che la soluzione funzioni quando viene copiata e incollata in un nuovo progetto.
Tech stack
rust
Dominio
documentationdeveloper experiencetesting
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust basicsCargo dev dependencies
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90