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
port(turbo): Link #3500
port(turbo): Link #3500
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@NicholasLYang is attempting to deploy a commit to the Vercel Team on Vercel. A member of the Team first needs to authorize it. |
34a4a34
to
7ee983d
Compare
7ee983d
to
79d9b7a
Compare
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.
We should rip out the Go impl with this PR. Would also really like to see some tests (not blocking though) since the integration test doesn't hit any of the logic inside link
to initialize one and only one copy of each config struct.
- Removed Go code for link - Print on error to match Go
Co-authored-by: Chris Olszewski <chrisdolszewski@gmail.com>
@@ -13,27 +13,38 @@ pub trait UserClient { | |||
fn set_token(&mut self, token: String); | |||
async fn get_user(&self) -> Result<UserResponse>; | |||
async fn get_teams(&self) -> Result<TeamsResponse>; | |||
async fn get_caching_status(&self, team_id: &str) -> Result<CachingStatus>; | |||
async fn get_caching_status(&self, team_id: &str) -> Result<CachingStatusResponse>; |
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.
Is it important that it returns the structure of the response and not just the data from the trait? e.g. returning CachingStatus
vs CachingStatusResponse
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.
It's not particularly important. At some point we might add pagination, at which point the response type will have more importance.
|
||
if !gitignore_path.exists() { | ||
let mut gitignore = File::create(gitignore_path)?; | ||
#[cfg(unix)] |
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.
Should we include setting the permissions on macos as well?
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.
Isn't macOS also unix?
Ported `link`. Required porting more `APIClient` functionality --------- Co-authored-by: Chris Olszewski <chrisdolszewski@gmail.com>
Ported
link
. Required porting moreAPIClient
functionality