Page MenuHomePhabricator
Differential D11435

[protos] Reorder some RPCs
ClosedPublic
Actions

Authored by bartek on Mar 28 2024, 3:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 9:38 AM
Unknown Object (File)
Thu, Dec 12, 8:50 PM
Unknown Object (File)
Nov 13 2024, 12:45 AM
Unknown Object (File)
Nov 12 2024, 10:39 PM
Unknown Object (File)
Oct 31 2024, 7:23 PM
Unknown Object (File)
Oct 31 2024, 5:36 PM
Unknown Object (File)
Oct 18 2024, 6:26 PM
Unknown Object (File)
Oct 18 2024, 6:26 PM
View All 59 Files
Subscribers
tomek

Details

Reviewers
ashoat
varun
Commits
rCOMMe4ead704d6b4: [protos] Reorder some RPCs
Summary

Minor cleanup in proto file structure:

  • Replaced // ... with /* ... */ for group headers to emphasize them over other comments
  • Moved some RPCs between groups
  • This also reordered generated code

I'm open to other comment suggestions as well. I prioritized these because of RPC order affecting codegen

Depends on D11379

Test Plan

N/A

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Mar 28 2024, 3:22 AM
bartek held this revision as a draft.
Herald added a subscriber: tomek. · View Herald TranscriptMar 28 2024, 3:22 AM
Harbormaster completed remote builds in B27757: Diff 38421.Mar 28 2024, 3:37 AM
bartek added a child revision: D11465: [protos] Add RPC to log in existing device.Mar 28 2024, 10:41 PM
bartek published this revision for review.Mar 28 2024, 11:05 PM
ashoat accepted this revision.Mar 29 2024, 10:47 AM

Seems reasonable

shared/protos/identity_unauth.proto
40–47 ↗(On Diff #38421)

Can we replace "Ashoat's" with "authoritative"?

This revision is now accepted and ready to land.Mar 29 2024, 10:47 AM
bartek updated this revision to Diff 38702.Apr 3 2024, 2:51 AM

Use 'authoritative' where applicable

Harbormaster completed remote builds in B27930: Diff 38702.Apr 3 2024, 3:10 AM
Closed by commit rCOMMe4ead704d6b4: [protos] Reorder some RPCs (authored by bartek). · Explain WhyApr 3 2024, 3:17 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMe4ead704d6b4: [protos] Reorder some RPCs.

Revision Contents
Changeset List

Diff 38708

View Options

shared/protos/identity_auth.proto

Loading...
View Options

shared/protos/identity_unauth.proto

Loading...
View Options

web/protobufs/identity-auth-client.cjs

Loading...
View Options

web/protobufs/identity-auth-client.cjs.flow

Loading...
View Options

web/protobufs/identity-unauth.cjs

Loading...
View Options

web/protobufs/identity-unauth.cjs.flow

Loading...