diesel-rs/diesel

First Rust example code on the website is unidiomatic

Open

#2451 aperta il 6 lug 2020

Vedi su GitHub
 (4 commenti) (1 reazione) (0 assegnatari)Rust (1003 fork)batch import
documentationhelp wanted

Metriche repository

Star
 (12.054 star)
Metriche merge PR
 (Merge medio 30g 16h) (30 PR mergiate in 30 g)

Descrizione

The first Rust code example, also available under examples/postgres/getting_started_step_1, is using .ok(); to discard the error case of a Result. If ignoring any errors is intentional, that line should be updated from

dotenv().ok();

to

let _ = dotenv();

Otherwise, the .ok() should be replaced by something else (maybe .expect("dotenv setup to be successful")).

Checklist

  • I have already looked over the issue tracker for similar issues.
  • This issue can be reproduced on Rust's stable channel. (Your issue will be closed if this is not the case) N/A

Guida contributor

Direzione di ricerca
Aggiorna il codice di esempio nel repository sostituendo `.ok()` con `let = dotenv();` o `.expect("...")` a seconda dei casi.
Tech stack
rust
Dominio
backend
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90