-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: initial working vscode web extension #155
Conversation
Deploying topology with Cloudflare Pages
|
3f6b316
to
84c2710
Compare
4c78194
to
4dbd0e9
Compare
This fixes the underlying issue that required a `skipLibCheck` in the first place.
1a9c220
to
fb5eb88
Compare
Offtopic: one day I should pick your brain on your pattern of using |
I believe the correct file path would be |
I like a linear history that tells a story, so will try to
Just to confirm, you're looking at https://github.dev/pi-base/data and clicking on the link that shows up in the markdown hover preview, right? (Or – if you cmd+click on the actual ID itself or use VSCode's "go to definition", are those working as expected for you?)
I think it's actually some arcane |
like 113e67d |
Yeah, working on debugging it now. I have no idea why it would just silently not show up in the web, but work fine locally. It may take be a bit to get set up with some kind of process for remote debugging; I'm going to move this back to draft for now. |
@StevenClontz – alright 53b3c7c should fix the links in hover text. The published extension seems to be working for me on github.dev but failing to activate entirely in a dedicated codespace. 🫠 Is that also what you're seeing? |
This was (squashed) merged in via #160. |
Partially addresses #5. There are a few cascading changes here that are related, but may not look like it:
vite
v4vite
to v5 implied upgrades tovitest
,sveltekit
vitest
had some breaking changes (snapshots, rejected promises)