Skip to content
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

cmd/tailscale/cli: remove special-casing of JS/WASM #5265

Closed
wants to merge 1 commit into from

Conversation

mihaip
Copy link
Contributor

@mihaip mihaip commented Aug 2, 2022

We're no longer compiling the entire CLI for WASM, so we can remove
some wrapper functions and GOOS checks.

Effectively reverts 75de4e9.
5df7ac7, eda647c
and other changes.

Fixes #5146

Signed-off-by: Mihai Parparita mihai@tailscale.com

We're no longer compiling the entire CLI for WASM, so we can remove
some wrapper functions and GOOS checks.

Effectively reverts 75de4e9.
5df7ac7, eda647c
and other changes.

Fixes #5146

Signed-off-by: Mihai Parparita <mihai@tailscale.com>
@mihaip mihaip requested a review from bradfitz August 2, 2022 18:29
@bradfitz
Copy link
Member

bradfitz commented Aug 2, 2022

Who says we don't want the full CLI in the future? :)

@mihaip
Copy link
Contributor Author

mihaip commented Aug 2, 2022

Fair enough. I can shelve this and clean up the direct fmt.Println calls that have crept in since the initial switch.

@mihaip mihaip closed this Aug 2, 2022
mihaip added a commit that referenced this pull request Aug 2, 2022
Since we're keeping the JS support in the CLI (see #5265 (comment)),
make sure it keeps building.

Signed-off-by: Mihai Parparita <mihai@tailscale.com>
mihaip added a commit that referenced this pull request Aug 2, 2022
Since we're keeping the JS support in the CLI (see #5265 (comment)),
make sure it keeps building.

Signed-off-by: Mihai Parparita <mihai@tailscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cmd/tsconnect: Remove JS references from CLI
2 participants