Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

chore: refactor tailnetAPIConnector to use dialer#15347

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
spikecurtis merged 1 commit intomainfromspike/14729-dialer
Nov 7, 2024

Conversation

@spikecurtis
Copy link
Contributor

@spikecurtisspikecurtis commentedNov 4, 2024•
edited
Loading

refactorstailnetAPIConnector to use theDialer interface intailnet, introduced lower in this stack of PRs. This will let us use the same Tailnet API handling code across different things that connect to the Tailnet API (CLI client, coderd, workspace proxies, and soon: Coder VPN).

chore re:#14729

@spikecurtisGraphite App
Copy link
ContributorAuthor

spikecurtis commentedNov 4, 2024•
edited
Loading

Copy link
Collaborator

@sreyasreya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Just some minor nits LGTM

@spikecurtisspikecurtisforce-pushed thespike/14729-resume-token-controller branch fromaf61bca to4f79cf7CompareNovember 5, 2024 09:51
@spikecurtisspikecurtisforce-pushed thespike/14729-resume-token-controller branch from4f79cf7 to5552b4fCompareNovember 5, 2024 10:02
@spikecurtisspikecurtisforce-pushed thespike/14729-resume-token-controller branch from5552b4f to2180c3fCompareNovember 6, 2024 13:13
@spikecurtisspikecurtisforce-pushed thespike/14729-resume-token-controller branch from2180c3f to82852adCompareNovember 6, 2024 16:05
@spikecurtisspikecurtisforce-pushed thespike/14729-resume-token-controller branch 2 times, most recently frome702cb7 to49baee8CompareNovember 6, 2024 16:24
@spikecurtisspikecurtis changed the base branch fromspike/14729-resume-token-controller tographite-base/15347November 7, 2024 07:32
@spikecurtisspikecurtis changed the base branch fromgraphite-base/15347 tomainNovember 7, 2024 07:33
@spikecurtisspikecurtis merged commit2d061e6 intomainNov 7, 2024
27 checks passed
@spikecurtisGraphite App
Copy link
ContributorAuthor

Merge activity

  • Nov 7, 8:24 AM EST: A user merged this pull request withGraphite.

@spikecurtisspikecurtis deleted the spike/14729-dialer branchNovember 7, 2024 13:24
jaaydenh pushed a commit that referenced this pull requestNov 8, 2024
refactors `tailnetAPIConnector` to use the `Dialer` interface in `tailnet`, introduced lower in this stack of PRs. This will let us use the same Tailnet API handling code across different things that connect to the Tailnet API (CLI client, coderd, workspace proxies, and soon: Coder VPN).chore re:#14729
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@sreyasreyasreya approved these changes

@ethanndicksonethanndicksonethanndickson approved these changes

Assignees

@spikecurtisspikecurtis

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@spikecurtis@sreya@ethanndickson
[8]ページ先頭
©2009-2025 Movatter.jp