Page MenuHomePhabricator

[services][blob] Initial commit for Rust service
ClosedPublic

Authored by bartek on Nov 18 2022, 8:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 23, 8:18 PM
Unknown Object (File)
Tue, Apr 23, 8:18 PM
Unknown Object (File)
Tue, Apr 23, 8:18 PM
Unknown Object (File)
Tue, Apr 23, 8:18 PM
Unknown Object (File)
Tue, Apr 23, 8:14 PM
Unknown Object (File)
Wed, Apr 3, 8:15 AM
Unknown Object (File)
Mar 21 2024, 8:22 PM
Unknown Object (File)
Mar 21 2024, 6:19 PM
Subscribers

Details

Summary

This is my first project in Rust and I am new to the language,
so I wrote a classic hello-world app ;)

Initialized a fresh Rust project and set up generating code from proto.

Depends on D5675

Test Plan
cd services/blob
cargo run

should build and display hello world

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Nov 18 2022, 9:20 AM
bartek added inline comments.
services/blob/Cargo.toml
7 ↗(On Diff #18576)

Should be 2021 - noticed that I didn't commit this typo fix here - it is fixed two diffs later

varun added inline comments.
services/blob/Cargo.toml
7 ↗(On Diff #18576)

I think it'd be better to just fix it here and rebase your stack

This revision is now accepted and ready to land.Nov 18 2022, 11:42 AM
services/blob/Cargo.toml
7 ↗(On Diff #18576)

it is fixed two diffs later

This is a big anti-pattern in the stacked diffs workflow. Unless it's extremely difficult, you should always aim to update the diff where the issue was introduced, rather than asking the reviewer to reference a later diff to see where it was fixed

services/blob/Cargo.toml
7 ↗(On Diff #18576)

Ok, fine. I'll do it here.

Accepting, but please update the edition before landing

Fix typo in cargo.toml edition field