google/comprehensive-rust

Include instructions for connecting to server in Tasks example

Open

#1 349 ouverte le 11 oct. 2023

Voir sur GitHub
 (6 commentaires) (0 réactions) (1 assigné)Rust (2 009 forks)batch import
good first issue

Métriques du dépôt

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

Description

Discussed in https://github.com/google/comprehensive-rust/discussions/1316

Originally posted by njr0 October 6, 2023 .. I made it run OK (by just adding tokio = { version = "1.32.0", features = ["full"] } as a dependency in Cargo.toml, and I could get it to say connection from 127.0.0.1:64382 etc. by using curl, but I guess I'd have to write code to connect to the socket and receive the question and send a reply and stuff. Presumably, that's what's in the "prepared main.rs. ..

nc or telnet are good tools for this. It's not HTTPS, so curl's not a good choice.

This should be mentioned in the speaker notes.

Guide contributeur

Direction de recherche
Cette demande propose d'ajouter des instructions dans les notes du conférencier pour l'exemple Tasks afin d'utiliser `nc` ou `telnet` pour se connecter au serveur, comme discuté dans la discussion liée. Le changement se trouve probablement dans les fichiers Markdown du matériel de cours, comme les notes du conférencier pour la section concernée. Le mainteneur doit ajouter une brève note expliquant que curl n'est pas adapté et que nc ou telnet peuvent être utilisés pour tester le serveur.
Stack technique
rust
Domaine
documentation
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.
1
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90