google/comprehensive-rust

Explicitly call out dev-dependencies used in solutions

Open

#1 290 ouverte le 2 oct. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Rust (32 954 stars) (2 009 forks)batch import
enhancementgood first issuehelp wanted

Description

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?

Guide contributeur

Stack technique
rust
Domaine
documentationdeveloper experiencetesting
Type d'issue
documentation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Rust basicsCargo dev dependencies
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
Investigate the solution file at exercises/day 3/solutions afternoon.html and its Cargo.toml. The issue references PR #1288, which may contain discussion. Consider adding a note in the solution text listing the required `cargo add` commands for dev dependencies like tempfile. Alternatively, evaluate splitting the shared Cargo.toml to include per solution dev dependencies. Ensure the solution works when copy pasted into a new project.