google/comprehensive-rust

Explicitly call out dev-dependencies used in solutions

Fechada

#1.290 aberto em 2 de out. de 2023

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Rust (2.009 forks)batch import
enhancementgood first issuehelp wanted

Métricas do repositório

Stars
 (32.954 estrelas)
Métricas de merge de PR
 (Mesclagem média 9d 23h) (36 fundiu PRs em 30d)

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?

Guia do colaborador

Direção de pesquisa
Investigue o arquivo de solução em exercises/day 3/solutions afternoon.html e seu Cargo.toml. A issue faz referência ao PR #1288, que pode conter discussões. Considere adicionar uma nota no texto da solução listando os comandos `cargo add` necessários para dev dependencies como tempfile. Alternativamente, avalie a divisão do Cargo.toml compartilhado para incluir dev dependencies por solução. Certifique se de que a solução funcione quando copiada e colada em um novo projeto.
Pilha de tecnologia
rust
Domain
documentationdeveloper experiencetesting
Tipo Issue
Documentação
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust basicsCargo dev dependencies
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90