google/comprehensive-rust

Explicitly call out dev-dependencies used in solutions

Fermée

#1 290 ouverte le 2 oct. 2023

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Rust (2 009 forks)batch import
enhancementgood first issuehelp wanted

Métriques du dépôt

Stars
 (32 954 étoiles)
Métriques de merge PR
 (Merge moyen 9j 23h) (36 PRs mergées en 30 j)

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

Direction de recherche
Examinez le fichier de solution à exercises/day 3/solutions afternoon.html et son Cargo.toml. L'issue fait référence à la PR #1288, qui peut contenir des discussions. Envisagez d'ajouter une note dans le texte de la solution listant les commandes `cargo add` nécessaires pour les dev dependencies comme tempfile. Alternativement, évaluez la possibilité de scinder le Cargo.toml partagé pour inclure les dev dependencies par solution. Assurez vous que la solution fonctionne lorsqu'elle est copiée collée dans un nouveau projet.
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.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust basicsCargo dev dependencies
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90