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

add some more logging to the extension startup #8047

Merged
merged 1 commit into from Apr 30, 2024
Merged

Conversation

arlyon
Copy link
Contributor

@arlyon arlyon commented Apr 26, 2024

Description

Adds a logging channel for the turbo vsc extension and rework some logging to prevent noisiness in the notification bar. This also adds some extra handling for homebrew based npm installs, and some additional handling to ensure paths are trimmed properly.

Testing Instructions

Build and boot up the extension and observe the new output tab for Turbo.

Closes TURBO-2894

Copy link

vercel bot commented Apr 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-basic-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 2:12pm
examples-native-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 2:12pm
examples-nonmonorepo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 2:12pm
examples-svelte-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 2:12pm
examples-vite-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 2:12pm
5 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 2:12pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 2:12pm
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 2:12pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 2:12pm
rust-docs ⬜️ Ignored (Inspect) Visit Preview Apr 30, 2024 2:12pm

Copy link
Contributor Author

arlyon commented Apr 26, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @arlyon and the rest of your teammates on Graphite Graphite

Copy link
Contributor

github-actions bot commented Apr 26, 2024

🟢 Turbopack Benchmark CI successful 🟢

Thanks

Copy link
Contributor

github-actions bot commented Apr 26, 2024

🟢 CI successful 🟢

Thanks

Copy link

socket-security bot commented Apr 26, 2024

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

packages/turbo-vsc/src/extension.ts Outdated Show resolved Hide resolved
Copy link
Contributor

✅ This change can build next-swc

@arlyon arlyon merged commit 903ce78 into main Apr 30, 2024
50 checks passed
@arlyon arlyon deleted the arlyon/lsp-logging branch April 30, 2024 14:16
Copy link
Contributor Author

arlyon commented Apr 30, 2024

Merge activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants