-
Notifications
You must be signed in to change notification settings - Fork 288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
apis: add UISession (a straight port of the old view.proto) #4525
Conversation
the only hand-written stuff is in uisession_types.go , everything else is generated |
TiltCloudTeamName string `json:"tiltCloudTeamName" protobuf:"bytes,6,opt,name=tiltCloudTeamName"` | ||
|
||
// TiltCloudSchemeHost reports the base URL of the Tilt Cloud instance | ||
// associated with this Tilt binary. Usually https://cloud.tilt.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// associated with this Tilt binary. Usually https://cloud.tilt.dev | |
// associated with this Tilt process. Usually https://cloud.tilt.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(this is mostly changed via env var when running tilt)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
// associated with this Tilt binary. Usually https://cloud.tilt.dev | ||
TiltCloudSchemeHost string `json:"tiltCloudSchemeHost" protobuf:"bytes,7,opt,name=tiltCloudSchemeHost"` | ||
|
||
// TiltCloudTeamID reports the unique team name if the user is signed into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// TiltCloudTeamID reports the unique team name if the user is signed into | |
// TiltCloudTeamID reports the unique team id if the user is signed into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Hello @milas, @landism,
Please review the following commits I made in branch nicks/uisession6:
4ec22ef (2021-05-06 15:15:34 -0400)
apis: add UISession (a straight port of the old view.proto)
Code review reminders, by giving a LGTM you attest that: